[llvm] [AMDGPU][DAG] Remove AssertZext before some intrinsics (PR #146052)
    Matt Arsenault via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Fri Jun 27 06:27:11 PDT 2025
    
    
  
================
@@ -3985,6 +3985,24 @@ SDValue AMDGPUTargetLowering::performAssertSZExtCombine(SDNode *N,
     }
   }
 
+  // AssertZext in front of these intrinsics is not necessary, the lowering of
+  // the intrinsics into a register read will insert one if it is needed.
----------------
arsenm wrote:
It is an assertion that doesn't generate code. If it does that's a bug. We should probably skip adding it from the original callsite if the known range is already implied by the intrinsic declaration 
https://github.com/llvm/llvm-project/pull/146052
    
    
More information about the llvm-commits
mailing list