[all-commits] [llvm/llvm-project] c65079: [Reassociate] Enable FP reassociation via 'reassoc...

[wjristow via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: c6507930493bf57d88bcb1f7d83da1b3c08ab02a
      https://github.com/llvm/llvm-project/commit/c6507930493bf57d88bcb1f7d83da1b3c08ab02a
  Author: Warren Ristow <warren.ristow at sony.com>
  Date:   2022-07-15 (Fri, 15 Jul 2022)

  Changed paths:
    M llvm/lib/Transforms/Scalar/Reassociate.cpp
    M llvm/test/Transforms/PhaseOrdering/fast-basictest.ll
    M llvm/test/Transforms/Reassociate/fast-basictest.ll

  Log Message:
  -----------
  [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

Reviewed By: spatel

Differential Revision: https://reviews.llvm.org/D129523