[PATCH] D25802: [AMDGPU] Promote ctlz (i1, i16] intrinsic to i32

Tom Stellard via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 20 08:40:32 PDT 2016


tstellarAMD added inline comments.


================
Comment at: lib/Target/AMDGPU/AMDGPUCodeGenPrepare.cpp:482
+
+  if (needsPromotionToI32(I.getType()))
+    Changed |= promoteCtlzToI32(I);
----------------
Shouldn't we only be doing this for uniform ops?  Otherwise the SelectionDAG should be able to do the promotion.


https://reviews.llvm.org/D25802





More information about the llvm-commits mailing list