This is an archive of the discontinued LLVM Phabricator instance.

Add branch emulation to aarch64 instruction emulator
ClosedPublic

Authored by tberghammer on Jun 24 2015, 11:37 AM.

Details

Summary

Add branch emulation to aarch64 instruction emulator

The emulation of the branches are required by the new stack
unwinding logic to reinstantiate the prologue at the right place.

Diff Detail

Repository
rL LLVM

Event Timeline

tberghammer retitled this revision from to Add branch emulation to aarch64 instruction emulator.
tberghammer updated this object.
tberghammer edited the test plan for this revision. (Show Details)
tberghammer added reviewers: jasonmolenda, clayborg.
tberghammer added a subscriber: Unknown Object (MLST).
emaste added a subscriber: emaste.Jun 24 2015, 12:25 PM
emaste added inline comments.
source/Plugins/Instruction/ARM64/EmulateInstructionARM64.cpp
880 ↗(On Diff #28377)

Shouldn't this be a comment rather than #if 0?

tberghammer added inline comments.Jun 24 2015, 12:30 PM
source/Plugins/Instruction/ARM64/EmulateInstructionARM64.cpp
880 ↗(On Diff #28377)

I did this based on the style we use in EmulateInstructionARM but it is definitely a comment. I can change it if we prefer that way

jasonmolenda edited edge metadata.Jun 24 2015, 8:22 PM

Greg's the best person to review this - as for the comment vrs. #if 0, I don't have a preference; I think for larger blocks of code the #if 0 is less noisy but it can also be easy to miss the fact that the code block is commented out when you're readingit.

clayborg accepted this revision.Jun 25 2015, 10:09 AM
clayborg edited edge metadata.

Looks good.

This revision is now accepted and ready to land.Jun 25 2015, 10:09 AM
This revision was automatically updated to reflect the committed changes.