[PATCH] D66050: Improve division estimation of floating points.

[Sanjay Patel via Phabricator via llvm-commits]

spatel added inline comments.


================
Comment at: llvm/test/CodeGen/PowerPC/combine-fneg.ll:19
+; CHECK-NEXT:    xxswapd 4, 4
 ; CHECK-NEXT:    xvnmsubadp 1, 2, 0
+; CHECK-NEXT:    xxlor 3, 2, 2
----------------
qiucf wrote:
> hfinkel wrote:
> > xbolva00 wrote:
> > > Regression
> > There's still an extra arithmetic instruction here?
> It's as expected. Since `visitFDIV` calls `combineRepeatedFPDivisors` to transform the vector divisions before `BuildDivEstimate`.
> 
> But the extra instructions are really redundant here. If `visitFMA` folds `(fma (fneg a) (fneg b) c)` into `(fma a b c)` like what `visitFMUL` does, the extra instructions will get eliminated. This can be done in future patch.
Did rL370071 solve that?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D66050