[llvm] f9f79d9 - [AMDGPU] Fix VTList for BUFFER_LOAD and DENORM_MODE (#138704)
    via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Tue May  6 11:02:43 PDT 2025
    
    
  
Author: Jay Foad
Date: 2025-05-06T19:02:39+01:00
New Revision: f9f79d9331975d9779b9eb9f77f133c8cf25080e
URL: https://github.com/llvm/llvm-project/commit/f9f79d9331975d9779b9eb9f77f133c8cf25080e
DIFF: https://github.com/llvm/llvm-project/commit/f9f79d9331975d9779b9eb9f77f133c8cf25080e.diff
LOG: [AMDGPU] Fix VTList for BUFFER_LOAD and DENORM_MODE (#138704)
This fixes bugs that would be caught by verifyTargetNode in #119709.
Added: 
    
Modified: 
    llvm/lib/Target/AMDGPU/SIISelLowering.cpp
Removed: 
    
################################################################################
diff  --git a/llvm/lib/Target/AMDGPU/SIISelLowering.cpp b/llvm/lib/Target/AMDGPU/SIISelLowering.cpp
index d7f1f1c651d68..b08b6b46fc52c 100644
--- a/llvm/lib/Target/AMDGPU/SIISelLowering.cpp
+++ b/llvm/lib/Target/AMDGPU/SIISelLowering.cpp
@@ -8703,7 +8703,7 @@ SDValue SITargetLowering::lowerSBuffer(EVT VT, SDLoc DL, SDValue Rsrc,
     LoadVT = MVT::getVectorVT(LoadVT.getScalarType(), 4);
   }
 
-  SDVTList VTList = DAG.getVTList({LoadVT, MVT::Glue});
+  SDVTList VTList = DAG.getVTList({LoadVT, MVT::Other});
 
   // Use the alignment to ensure that the required offsets will fit into the
   // immediate offsets.
@@ -11184,9 +11184,10 @@ SDValue SITargetLowering::LowerFDIV32(SDValue Op, SelectionDAG &DAG) const {
       const SDValue DisableDenormValue = getSPDenormModeValue(
           FP_DENORM_FLUSH_IN_FLUSH_OUT, DAG, Info, Subtarget);
 
+      SDVTList BindParamVTs = DAG.getVTList(MVT::Other, MVT::Glue);
       DisableDenorm =
-          DAG.getNode(AMDGPUISD::DENORM_MODE, SL, MVT::Other, Fma4.getValue(1),
-                      DisableDenormValue, Fma4.getValue(2))
+          DAG.getNode(AMDGPUISD::DENORM_MODE, SL, BindParamVTs,
+                      Fma4.getValue(1), DisableDenormValue, Fma4.getValue(2))
               .getNode();
     } else {
       assert(HasDynamicDenormals == (bool)SavedDenormMode);
        
    
    
More information about the llvm-commits
mailing list