[llvm] [RISCV] Force relocations if initial MCSubtargetInfo contains FeatureRelax (PR #77436)
via llvm-commits
llvm-commits at lists.llvm.org
Tue Jan 9 02:03:03 PST 2024
github-actions[bot] wrote:
<!--LLVM CODE FORMAT COMMENT: {clang-format}-->
:warning: C/C++ code formatter, clang-format found issues in your code. :warning:
<details>
<summary>
You can test this locally with the following command:
</summary>
``````````bash
git-clang-format --diff daecc303bb719ed63566fcb343afec169826f82c d91936f761da3efc61086e693b48e26b1926cf6b -- llvm/lib/Target/RISCV/MCTargetDesc/RISCVELFStreamer.cpp
``````````
</details>
<details>
<summary>
View the diff from clang-format here.
</summary>
``````````diff
diff --git a/llvm/lib/Target/RISCV/MCTargetDesc/RISCVELFStreamer.cpp b/llvm/lib/Target/RISCV/MCTargetDesc/RISCVELFStreamer.cpp
index 961b8f0afe..880f469f03 100644
--- a/llvm/lib/Target/RISCV/MCTargetDesc/RISCVELFStreamer.cpp
+++ b/llvm/lib/Target/RISCV/MCTargetDesc/RISCVELFStreamer.cpp
@@ -40,8 +40,8 @@ RISCVTargetELFStreamer::RISCVTargetELFStreamer(MCStreamer &S,
// `j label` in `.option norelax; j label; .option relax; ...; label:` needs a
// relocation to ensure the jump target is correct after linking. This is due
// to a limitation that shouldForceRelocation has to make the decision upfront
- // without knowing a possibly future .option relax. When RISCVAsmParser is used,
- // its ParseInstruction may call setForceRelocs as well.
+ // without knowing a possibly future .option relax. When RISCVAsmParser is
+ // used, its ParseInstruction may call setForceRelocs as well.
if (STI.hasFeature(RISCV::FeatureRelax))
static_cast<RISCVAsmBackend &>(MAB).setForceRelocs();
}
``````````
</details>
https://github.com/llvm/llvm-project/pull/77436
More information about the llvm-commits
mailing list