[PATCH] D155953: [RISCV][MC] Add CLI option to disable branch relaxation
Job Noorman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 24 00:15:55 PDT 2023
jobnoorman updated this revision to Diff 543400.
jobnoorman added a comment.
Address reviewer comments:
- Make option `cl::Hidden`
- Rename option to `-riscv-asm-relax-branches`
- Add tests for branches to defined symbols
The tests revealed some issues for the case where symbols are out of range for
compressed jumps and branches: no errors were reported because it was impossible
to trigger this (due to relaxation). This update also adds to necessary error
reporting.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D155953/new/
https://reviews.llvm.org/D155953
Files:
llvm/lib/Target/RISCV/MCTargetDesc/RISCVAsmBackend.cpp
llvm/test/MC/RISCV/long-jump-disable-relax.s
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D155953.543400.patch
Type: text/x-patch
Size: 3534 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230724/6c1e2f0a/attachment.bin>
More information about the llvm-commits
mailing list