[llvm] [SystemZ] Don't lower float/double ATOMIC_[LOAD|STORE] to [LOAD|STORE] (PR #75879)
Ulrich Weigand via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 13 13:17:42 PDT 2024
================
@@ -6587,6 +6574,27 @@ SDValue SystemZTargetLowering::combineTruncateExtract(
return SDValue();
}
+// Replace ALoad with a new ATOMIC_LOAD with a result that is extended to VT
+// per ETy.
+static SDValue extendAtomicLoad(AtomicSDNode *ALoad, EVT VT, SelectionDAG &DAG,
+ ISD::LoadExtType ETy) {
----------------
uweigand wrote:
Given that this patch adds extending atomic loads as a common-code feature, shouldn't now also common code attempt to match them in DAGCombine? Why does this need to be platform-specific? But I guess this can be looked at in a separate patch.
https://github.com/llvm/llvm-project/pull/75879
More information about the llvm-commits
mailing list