[PATCH] D102535: [RISCV] -mno-relax: emit .option norelax

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added a comment.

In D102535#2779469 <https://reviews.llvm.org/D102535#2779469>, @kito-cheng wrote:

> GCC pass `-mno-relax` to assembler now: https://gcc.gnu.org/git/?p=gcc.git;a=commit;h=3b0a7d624e64eeb81e4d5e8c62c46d86ef521857

I think an assembler only option should be spelled as `-Wa,*`. The driver option isn't useful and should just be avoided.

For `-mno-relax`, I think there can be arguments that the assembler generation may be affected (to appease assembler, for example) so a driver option is needed.

If the driver option `-mno-relax` is still preferred over `-Wa,-mno-relax`, I can abandon this and submit another one passing `-mno-relax` to GNU as for `-fno-integrated-as`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102535