Page MenuHomeFreeBSD

arm: Remove armv5 supprt
ClosedPublic

Authored by imp on Feb 13 2023, 8:06 PM.
Tags
None
Referenced Files
F150208823: D38563.id117219.diff
Mon, Mar 30, 6:59 AM
Unknown Object (File)
Mon, Mar 30, 2:09 AM
Unknown Object (File)
Mon, Mar 30, 1:47 AM
Unknown Object (File)
Mon, Mar 30, 1:47 AM
Unknown Object (File)
Mon, Mar 30, 1:46 AM
Unknown Object (File)
Thu, Mar 19, 5:24 PM
Unknown Object (File)
Wed, Mar 4, 8:39 PM
Unknown Object (File)
Wed, Mar 4, 9:03 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 49747
Build 46637: 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.