[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