[llvm] [RISCV][ISEL] Lowering to load-acquire/store-release for RISCV Zalasr (PR #82914)
via llvm-commits
llvm-commits at lists.llvm.org
Sun Feb 25 02:14:07 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 a8c3b3e20db01d2947dbe87d0c557150ed777865 0e3ec6ab9873a8cccee2e39919186395eddeffc9 -- llvm/lib/Target/RISCV/RISCVISelLowering.cpp llvm/lib/Target/RISCV/RISCVISelLowering.h
``````````
</details>
<details>
<summary>
View the diff from clang-format here.
</summary>
``````````diff
diff --git a/llvm/lib/Target/RISCV/RISCVISelLowering.cpp b/llvm/lib/Target/RISCV/RISCVISelLowering.cpp
index d1829903a0..b6bd6097e8 100644
--- a/llvm/lib/Target/RISCV/RISCVISelLowering.cpp
+++ b/llvm/lib/Target/RISCV/RISCVISelLowering.cpp
@@ -20842,9 +20842,12 @@ bool RISCVTargetLowering::shouldInsertFencesForAtomic(
// will just not insert any fences
auto LI = dyn_cast<LoadInst>(I);
auto SI = dyn_cast<StoreInst>(I);
- if ((LI && (LI->getOrdering() == AtomicOrdering::SequentiallyConsistent))
- || (SI && (SI->getOrdering() == AtomicOrdering::SequentiallyConsistent))) {
- // Here, this is a load or store which is seq_cst, and needs a .aq or .rl
+ if ((LI &&
+ (LI->getOrdering() == AtomicOrdering::SequentiallyConsistent)) ||
+ (SI &&
+ (SI->getOrdering() == AtomicOrdering::SequentiallyConsistent))) {
+ // Here, this is a load or store which is seq_cst, and needs a .aq or
+ // .rl
// therefore we shouldn't try to insert fences
return false;
}
``````````
</details>
https://github.com/llvm/llvm-project/pull/82914
More information about the llvm-commits
mailing list