[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