[llvm] [RuntimeDyldChecker][AArch32] Add a PC offset to next_PC for ARM targets (PR #91746)

Stefan Gränitz via llvm-commits llvm-commits at lists.llvm.org
Tue May 14 05:32:51 PDT 2024


https://github.com/weliveindetail commented:

Hey! Thanks for your proposal. I think it's valuable. Your [second patch](https://github.com/llvm/llvm-project/pull/91746/commits/0eecc0ab808b592ccd41aadc32b91a55d9f177d9) seemed very much to the point. Maybe with a more verbose comment like: https://github.com/llvm/llvm-project/blob/142c89c381bccba1c688a90d7e9df91eaf78d4e3/llvm/test/ExecutionEngine/JITLink/AArch32/ELF_static_arm_reloc.s#L16-L17 We could then clean up these comments in the tests. What do you think?

https://github.com/llvm/llvm-project/pull/91746


More information about the llvm-commits mailing list