[llvm] [AMDGPU][DAG] Remove AssertZext before some intrinsics (PR #146052)

Jay Foad via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 27 04:09:51 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.
----------------
jayfoad wrote:

I don't understand. AssertZext is just an assertion, it does not generate any code, so removing it is presumably just working around some other part of the backend that doesn't know to look through it?

https://github.com/llvm/llvm-project/pull/146052


More information about the llvm-commits mailing list