[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