[PATCH] D128591: Transforms: Relax restrictions on pow(x, y) expansion

Paul Osmialowski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 28 09:27:15 PDT 2022


pawosm01 marked 2 inline comments as done.
pawosm01 added inline comments.


================
Comment at: llvm/test/Transforms/InstCombine/pow-4.ll:65
+;
+  %1 = call reassoc nnan ninf nsz double @llvm.pow.f64(double %x, double 4.000000e+00)
+  ret double %1
----------------
david-arm wrote:
> nit: Can you remove nsz here too?
Si.


================
Comment at: llvm/test/Transforms/InstCombine/pow-4.ll:125
+;
+  %1 = call reassoc nnan ninf nsz float @llvm.pow.f32(float %x, float -1.900000e+01)
+  ret float %1
----------------
david-arm wrote:
> nit: Can you remove nsz here too?
Si.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D128591/new/

https://reviews.llvm.org/D128591



More information about the llvm-commits mailing list