[PATCH] D155953: [RISCV][MC] Add CLI option to disable branch relaxation

[Alex Bradbury via Phabricator via llvm-commits]

asb added a comment.

I'm supportive of adding a targeted flag to make it easier to write these tests, with the idea that a more public facing flag might later be introduced that alters a full "strict" mode.

I think it might make sense to pass `cl::Hidden` to DisableBranchRelaxation so this doesn't become a documented feature.



================
Comment at: llvm/test/MC/RISCV/long-jump-disable-relax.s:2
+# RUN: llvm-mc -filetype=obj -triple=riscv64 -mattr=+c \
+# RUN:       --riscv-disable-branch-relaxation %s \
+# RUN:     | llvm-objdump -dr -M no-aliases - \
----------------
Nit: I think we normally use the `-single-hyphen-prefix` spelling for llvm-mc opts.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D155953