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

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 13 08:18:32 PST 2017


hfinkel added a comment.

In https://reviews.llvm.org/D28675#645313, @mehdi_amini wrote:

> What about individual FMF?


I agree; these transforms rely on the associativity of addition. As Mehdi says, you should check the FMFs on the individual instructions.


https://reviews.llvm.org/D28675





More information about the llvm-commits mailing list