[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 06:44:17 PST 2017

spatel added a comment.

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

> Obviously it'd be even easier to skip the getFlags() thing entirely now, since it can never evaluate to true. But the right way forward is to extend support for FMF, so...

Right - I think your previous suggestion to return a default flags object will work.


More information about the llvm-commits mailing list