Page MenuHomeFreeBSD

arm64: Fix a logic bug in is_load_instr().
ClosedPublic

Authored by jhb on Sep 13 2021, 6:30 PM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 14 2024, 5:48 PM
Unknown Object (File)
Feb 7 2024, 2:48 AM
Unknown Object (File)
Dec 22 2023, 11:47 PM
Unknown Object (File)
Sep 29 2023, 8:47 PM
Unknown Object (File)
Aug 13 2023, 2:10 AM
Unknown Object (File)
Jul 15 2023, 10:35 PM
Unknown Object (File)
Jul 6 2023, 6:35 PM
Unknown Object (File)
Jun 28 2023, 8:29 PM
Subscribers

Details

Summary

Logical and ('&&') was used to join two conditions instead of logical
or ('||') causing some store instructions to not be recognized.

Reported by: GCC 9 -Wparentheses

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jhb requested review of this revision.Sep 13 2021, 6:30 PM

I wonder if we need this. The store macro looks broken, they are only used in ddb to count instructions, and are defined as 0 on most architectures.

This revision is now accepted and ready to land.Sep 15 2021, 9:12 AM

I have no idea if this is needed or not TBH.

This revision was automatically updated to reflect the committed changes.