[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