[PATCH] D28675: [DAGCombine] require UnsafeFPMath for re-association of addition

[Nicolai Hähnle via Phabricator via llvm-commits]

nhaehnle created this revision.
nhaehnle added reviewers: spatel, hfinkel, mehdi_amini, arsenm.
nhaehnle added a subscriber: llvm-commits.
Herald added subscribers: wdng, nemanjai, jholewinski.
Herald added a reviewer: tstellarAMD.

The affected transforms all implicitly use associativity of addition,
for which we usually require unsafe math to be enabled.

The "Aggressive" flag is only meant to convey information about the
performance of the fused ops relative to a fmul+fadd sequence.

Fixes Bug 31626.


https://reviews.llvm.org/D28675

Files:
  lib/CodeGen/SelectionDAG/DAGCombiner.cpp
  test/CodeGen/AMDGPU/fma-combine.ll
  test/CodeGen/AMDGPU/mad-combine.ll
  test/CodeGen/NVPTX/fma-assoc.ll
  test/CodeGen/PowerPC/fma-assoc.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D28675.84288.patch
Type: text/x-patch
Size: 25155 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170113/d6c4729d/attachment.bin>