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

Evandro Menezes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 23 14:59:40 PDT 2021


evandro added a comment.

In D102535#2819032 <https://reviews.llvm.org/D102535#2819032>, @luismarques wrote:

> I thought I understood your point initially, but now I'm not sure.  Can you clarify what flexibility you're losing? Doesn't the emission of this directive reflect exactly the build settings you specify at compilation time? Why is it important to override this later?

If consistency is desired from `clang` to `as`, then `-Wa,-mno-relax` is preferred.  This way, the same assembly source can be assembled with or without relaxation without changing the source code.  Moreover, in the case of an assembly source file, the choice to relax its code or not can be made at build time, or, more specifically, at link time.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102535



More information about the llvm-commits mailing list