[llvm] [RFC][InstCombine] Combine fence with target specific sync scope (PR #93939)
Sameer Sahasrabuddhe via llvm-commits
llvm-commits at lists.llvm.org
Sun Jun 2 22:28:38 PDT 2024
================
@@ -30,6 +30,15 @@ define void @test_target_dependent_scope() {
ret void
}
+; CHECK-LABEL: combine_target_dependent_scope
+; CHECK-NEXT: fence syncscope("agent") seq_cst
+; CHECK-NEXT: ret void
+define void @combine_target_dependent_scope() {
+ fence syncscope("agent") acq_rel
+ fence syncscope("agent") seq_cst
+ ret void
+}
+
----------------
ssahasra wrote:
While it's safe to ignore and drop all MMRA, the ideal thing to do is compute the correct MMRA for the final fence. I don't know that is already happening in this path of InstCombine.
@Pierre-vh
https://github.com/llvm/llvm-project/pull/93939
More information about the llvm-commits
mailing list