Page MenuHomeFreeBSD

arm: Remove armv5 supprt
ClosedPublic

Authored by imp on Feb 13 2023, 8:06 PM.
Tags
None
Referenced Files
F167543911: D38563.id117119.diff
Sat, Aug 22, 2:36 PM
F167504883: D38563.id117120.diff
Sat, Aug 22, 8:52 AM
F167504517: D38563.id117219.diff
Sat, Aug 22, 8:50 AM
Unknown Object (File)
Fri, Aug 21, 12:27 PM
Unknown Object (File)
Fri, Aug 14, 3:56 PM
Unknown Object (File)
Mon, Aug 10, 10:54 AM
Unknown Object (File)
Mon, Aug 10, 10:46 AM
Unknown Object (File)
Mon, Aug 10, 10:45 AM

Details

Summary

This can only be true on armv5, so we can GC it.

Sponsored by: Netflix

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 49746
Build 46636: arc lint + arc unit

Event Timeline

imp requested review of this revision.Feb 13 2023, 8:06 PM
This revision is now accepted and ready to land.Feb 13 2023, 8:08 PM
imp retitled this revision from arm: Cleanup overly complicated expression to arm: Remove armv5 supprt.Feb 13 2023, 8:12 PM
imp edited the summary of this revision. (Show Details)

Read conditional backwards

This revision now requires review to proceed.Feb 13 2023, 8:12 PM

MACHINE_ARCH will always be armv6 or armv7, so expression never true

This revision is now accepted and ready to land.Feb 13 2023, 8:14 PM
fuz added a subscriber: fuz.

LGTM

This revision was automatically updated to reflect the committed changes.