[PATCH] D135960: [RISCV] Allow LI with symbol difference as constant

Alex Bradbury via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 8 02:08:46 PST 2023


asb added a comment.

Could you please update the patch description to say "Fixes #57461 <https://github.com/llvm/llvm-project/issues/57461>" or similar?

I think all the review comments are addressed and I don't have additional feedback - the only open line of discussion I think was about how `fixup_riscv_12_i` is the only fixup that can never result in a relocation (as pointed out by @jrtc27). That's true, but I also can't see an alternative way of handling it or any real disadvantage of this - it logically _is_ a fixup. Do you see an alternative approach, or was it more that you think it would make sense to adjust the naming so it's clearer it doesn't have a matching relocation?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D135960/new/

https://reviews.llvm.org/D135960



More information about the llvm-commits mailing list