[PATCH] D25069: Fix PREL31 relocation on ARM

Peter Smith via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 30 12:25:52 PDT 2016


peter.smith added a comment.

Your solution is the same way that R_ARM_PREL31 is handled in lld, so this looks fine from a correctness point of view.

I'm not familiar with this area so I think you'll need approval from a maintainer, they may also insist on a test being written.


https://reviews.llvm.org/D25069





More information about the llvm-commits mailing list