Skip to content

Commit

Permalink
[AMDGPU] Remove old isCheapToSpeculateCttz FIXME
Browse files Browse the repository at this point in the history
As confirmed on D132520 - this should always return true
  • Loading branch information
RKSimon committed Aug 24, 2022
1 parent 79f99bf commit 3cf4896
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion llvm/lib/Target/AMDGPU/AMDGPUISelLowering.cpp
Expand Up @@ -692,7 +692,6 @@ bool AMDGPUTargetLowering::isLoadBitCastBeneficial(EVT LoadTy, EVT CastTy,
// SI+ has instructions for cttz / ctlz for 32-bit values. This is probably also
// profitable with the expansion for 64-bit since it's generally good to
// speculate things.
// FIXME: These should really have the size as a parameter.
bool AMDGPUTargetLowering::isCheapToSpeculateCttz() const {
return true;
}
Expand Down

0 comments on commit 3cf4896

Please sign in to comment.