[PATCH] D158904: AMDGPU: Don't require all flags to expand fast powr
Stanislav Mekhanoshin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Aug 25 16:18:06 PDT 2023
rampitec accepted this revision.
rampitec added inline comments.
This revision is now accepted and ready to land.
================
Comment at: llvm/lib/Target/AMDGPU/AMDGPULibCalls.cpp:802
- // No unsafe math , no constant argument, do nothing
- if (!isUnsafeMath(FPOp) && !CF && !CINT && !CZero)
- return false;
----------------
rampitec wrote:
> I do not see why CF, CINT, and CZero cannot be null.
Ah, OK, it seems to be checked with every use.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D158904/new/
https://reviews.llvm.org/D158904
More information about the llvm-commits
mailing list