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

Warren Ristow via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 11 16:14:40 PDT 2022


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>


More information about the llvm-commits mailing list