Since the issue with trap value was fixed in D158191, tests now should pass on both platforms
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
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. |
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.