[PATCH] D62993: [PowerPC] Emit scalar min/max instructions with unsafe fp math

Nemanja Ivanovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 23 21:47:57 PDT 2019


nemanjai marked an inline comment as done.
nemanjai added inline comments.


================
Comment at: lib/Target/PowerPC/PPCInstrVSX.td:1267
+
+let AddedComplexity = 1, Predicates = [HasVSX] in {
+  def : Pat<(f32 (fminnum_ieee f32:$A, f32:$B)),
----------------
steven.zhang wrote:
> Just out of curious, why we set the complexity as 1 here instead of 400. 
Ha ha... because it's a bug. I'll change it to 400 on the commit.
It doesn't change semantics because there isn't an equivalent FPU choice to be made here, but it should still be consistent.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D62993





More information about the llvm-commits mailing list