[PATCH] D129523: [Reassociate] Enable FP reassociation via 'reassoc' and 'nsz'

[Warren Ristow via Phabricator via llvm-commits]

wristow created this revision.
wristow added reviewers: spatel, RKSimon, lebedev.ri.
Herald added a subscriber: hiraditya.
Herald added a project: All.
wristow requested review of this revision.
Herald added a project: LLVM.

[Reassociate] Enable FP reassociation via 'reassoc' and 'nsz'

Compiling with '-ffast-math' tuns on all the FastMathFlags (FMF), as
expected, and that enables FP reassociation.  Only the two FMF flags
'reassoc' and 'nsz' are technically required to perform reassociation,
but disabling other unrelated FMF bits is needlessly suppressing the
optimization.

This patch fixes that needless suppression, and makes appropriate
adjustments to test-cases, fixing some outstanding TODOs in the process.

Fixes: #56483


https://reviews.llvm.org/D129523

Files:
  llvm/lib/Transforms/Scalar/Reassociate.cpp
  llvm/test/Transforms/PhaseOrdering/fast-basictest.ll
  llvm/test/Transforms/Reassociate/fast-ReassociateVector.ll
  llvm/test/Transforms/Reassociate/fast-basictest.ll
  llvm/test/Transforms/Reassociate/pr42349.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D129523.443783.patch
Type: text/x-patch
Size: 17460 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220711/243d3de9/attachment.bin>