This is an archive of the discontinued LLVM Phabricator instance.

[BOLT][test] Enable exceptions_split tests for AArch64
ClosedPublic

Authored by Elvina on Aug 25 2023, 2:45 PM.

Details

Summary

Since the issue with trap value was fixed in D158191, tests now should pass on both platforms

Diff Detail

Event Timeline

Elvina created this revision.Aug 25 2023, 2:45 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 25 2023, 2:45 PM
Elvina requested review of this revision.Aug 25 2023, 2:45 PM
maksfb added inline comments.Aug 29 2023, 10:43 AM
bolt/test/runtime/pie-exceptions-split.test
19

Since you are enabling the test case for *all* platforms, how can you be sure to match all possible variants of the call instruction, i.e. beyond callq and bl? Perhaps, we can match just "printf" without the instruction? Alternatively, you can enable the test for a subset of currently supported architectures.

Otherwise, LGTM.

Elvina updated this revision to Diff 554811.Aug 30 2023, 12:55 PM
Elvina marked an inline comment as done.

updated the diff to check for printf only

maksfb accepted this revision.Aug 31 2023, 10:14 AM
This revision is now accepted and ready to land.Aug 31 2023, 10:14 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptSep 1 2023, 12:48 AM