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

[Saleem Abdulrasool via Phabricator via llvm-commits]

compnerd added a comment.

Hmm, actually looking at gcc, it appears that this is actually matching semantics with gcc:
https://github.com/riscv/riscv-gcc/blob/03cb20e5433cd8e65af6a1a6baaf3fe4c72785f6/gcc/config/riscv/riscv.c#L4590-L4593

In D102535#2769537 <https://reviews.llvm.org/D102535#2769537>, @jrtc27 wrote:

> In D102535#2769528 <https://reviews.llvm.org/D102535#2769528>, @compnerd wrote:
>
>> If the user is passing along `-mno-relax` encoding that in the object file isn't expensive, and it doesn't harm anything.  In fact, the user _could_ at a later point pass in `-mrelax` to the assembler to override the beahviour (with a warning).  But what @raj.khem states about the non-IAS case should hold - `-mno-relax` should get passed along to the assembler along with the hint in the object file.
>
> `.option norelax` overrides -mrelax, just like any other assembler directive.

Ah right, the attributes are processed late in ELF.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102535