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

[Alex Bradbury via Phabricator via llvm-commits]

asb added a comment.

Thanks for the updates Lewis. I think this is a good solution to the problem. I need to step through the code in more detail to give a final LGTM, but I think it resolves our questions.

Could you rename this to remove the WIP from the title?

The only potential issue is whether there is a user expectation that a file that starts with `.option norelax` should be assembled in the same way as passing norelax to the assembler. If relaxation is the default (which is the intention), then it won't. I'm just noting this for the record and in case anyone has a particular opinion on the matter, I don't think it's a blocking issue.


Repository:
  rL LLVM

https://reviews.llvm.org/D46423