[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.
https://reviews.llvm.org/D28675
More information about the llvm-commits
mailing list