[PATCH] D153346: [BOLT][RISCV] Add tests for simple CFG transformations

Rafael Auler via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 20 18:18:53 PDT 2023


rafauler added inline comments.


================
Comment at: bolt/test/RISCV/internal-func-reloc.s:12
+  /// These options are only used to make the assembler output easier to predict
+  .option norelax
+  .option norvc
----------------
This is something we might want to ask of binaries that intend to be BOLTed, as BOLT is going to do a lot of things that RISCV is attempting to do with linkers and linker relaxation. This is specially true if BOLT later reorders functions in a way that now requires thunks/stubs insertion to extend the range of branches that got prematurely relaxed by the linker.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153346



More information about the llvm-commits mailing list