[PATCH] D46423: [WIP, RISCV] Support .option relax and .option norelax

Alex Bradbury via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 15 07:45:02 PDT 2018


asb added a comment.
Herald added subscribers: PkmX, the_o, brucehoult, MartinMosbeck, rogfer01, jrtc27.

Are you planning on updating this? I still feel that either producing an error, or doing what the user says and honouring .option relax/norelax is the best option. Having .option relax 'sticky' for symbol differences slightly reduces the amount of broken code if a user switches between relax/norelax in the same section, but you still have the potential for calls or branches being resolved incorrectly don't you?


Repository:
  rL LLVM

https://reviews.llvm.org/D46423





More information about the llvm-commits mailing list