[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