[PATCH] D88066: [InstCombine] For pow(x, +/-0.5), stop falling into pow(x, 1.5), etc. case

Hubert Tong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 22 06:42:43 PDT 2020


hubert.reinterpretcast added inline comments.


================
Comment at: llvm/test/Transforms/InstCombine/pow-4.ll:240-241
+; SQRT-LABEL: @shrink_pow_libcall_half(
+; SQRT-NEXT:    [[SQRTF:%.*]] = call fast float @sqrtf(float [[X:%.*]])
+; SQRT-NEXT:    [[SQRTF1:%.*]] = call fast float @sqrtf(float [[X]])
+; SQRT-NEXT:    ret float [[SQRTF1]]
----------------
spatel wrote:
> That looks like another bug. We formed an extra sqrtf() call from a not-quite-dead pow() call?
I'm not sure I'll be able to chase this one. Do you have a recommendation on how to mark/track it?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88066



More information about the llvm-commits mailing list