[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