Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
llvm/test/CodeGen/AMDGPU/GlobalISel/cvt_f32_ubyte.ll | ||
---|---|---|
1126–1136 | This is unfortunate. The problem is that when CTLZ is expanded using FFBH, AMDGPUPostLegalizerCombinerHelper::matchUCharToFloat can no longer see that CTLZ of the high half of %masked = and i64 %arg0, 255 is known to be 32. It seems like we would need a whole bunch of extra constant folds and/or known bits logic to make this work again. |
@foad Coverity is complaining that you've repeated the 'Opc == AMDGPU::G_CTLZ_ZERO_UNDEF' check (which is dead code) - should the second one be 'Opc == AMDGPU::G_CTTZ_ZERO_UNDEF' ?