diff options
author | Job Noorman <jnoorman@igalia.com> | 2023-07-28 10:27:32 +0200 |
---|---|---|
committer | Job Noorman <jnoorman@igalia.com> | 2023-07-28 10:42:05 +0200 |
commit | afb2e9f44c13101c7b22c99e080b0beb0cd5b01e (patch) | |
tree | f93545cb35e0dfe9da76581e2d29339ec9c4f44f /llvm/lib/Object | |
parent | e76304d05e56f593a45fd54115e2d1d30a329a11 (diff) | |
download | llvm-afb2e9f44c13101c7b22c99e080b0beb0cd5b01e.zip llvm-afb2e9f44c13101c7b22c99e080b0beb0cd5b01e.tar.gz llvm-afb2e9f44c13101c7b22c99e080b0beb0cd5b01e.tar.bz2 |
[RISCV][MC] Add CLI option to disable branch relaxation
D154958 enables branch relaxation for unresolved symbols. This has an
interesting consequence for some LLD tests: branch relocations are
tested by using branches to undefined symbols and defining them, with
different values, on the LLD command line. These tests broke and there
doesn't seem to be an easy workaround: as far as I can tell, there is no
way to convince llvm-mc to emit a branch relocation to an undefined
symbol without branch relaxation kicking in.
This patch proposes to add a flag, `-riscv-asm-relax-branches=0`, to do
just that. The main purpose for this flag is for testing but it might be
seen as a first step to some kind of "strict" or WYSIWYG mode (i.e.,
what you give to the assembler is exactly what comes out). The need for
this has been mentioned in, for example, D108961. However, I suspect
there will be a lot of discussion around what exactly such a strict mode
would look like. Therefore, I gated this feature behind a CLI flag
instead of adding a new target feature.
Reviewed By: asb, MaskRay
Differential Revision: https://reviews.llvm.org/D155953
Diffstat (limited to 'llvm/lib/Object')
0 files changed, 0 insertions, 0 deletions