[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 27 12:15:58 PST 2017


hfinkel added a comment.

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

> > But you can't because they'll crash [...]
>
> That's what the hunk in SelectionDAG.cpp is about.


OIC, alright. I recommend that, if you want this to go into the branch, we do a targeted fix and then expand on that in follow-up.

> And d'oh on the SDValue thing, I'll fix that.




https://reviews.llvm.org/D28675





More information about the llvm-commits mailing list