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

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 20 07:19:56 PST 2017


spatel added a comment.

In https://reviews.llvm.org/D28675#651549, @nhaehnle wrote:

> Added the DefaultFlags. It does fix @spatel's example for me. Anything else?


That's all from me (someday, I'll get back to FMF in the DAG...), but I think someone who knows one of the affected backends should provide the approval.


https://reviews.llvm.org/D28675





More information about the llvm-commits mailing list