[PATCH] D67351: [InstCombine] Use SimplifyFMulInst to simplify multiply in fma.

[Florian Hahn via Phabricator via llvm-commits]

fhahn created this revision.
fhahn added reviewers: spatel, anemet, lebedev.ri.
Herald added a subscriber: hiraditya.
Herald added a project: LLVM.
fhahn updated this revision to Diff 219329.
fhahn added a comment.

Remove unrelated test-case.


This allows us to fold fma's that multiply with 0.0. Also, the
multiply by 1.0 case is handled there as well. The fneg/fabs cases
are not handled by SimplifyFMulInst, so we need to keep them.
I am not sure what the reason for not having them in SimplifyFMulInst
is.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D67351

Files:
  llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp
  llvm/test/Transforms/InstCombine/fma.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D67351.219329.patch
Type: text/x-patch
Size: 5021 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190909/a3ece8a6/attachment.bin>