[PATCH] propagate IR-level fast-math-flags to DAG nodes

[Eric Christopher]

Absolutely agreed. Thanks!

-eric

On Tue, Apr 28, 2015 at 8:41 AM Sanjay Patel <spatel at rotateright.com> wrote:

> In http://reviews.llvm.org/D8900#162327, @echristo wrote:
>
> > It's still my preference if possible because it'll take the API nfc
> change out of the mix on the patch. Is it possible for you to do that?
>
>
> Hi Eric -
>
> Sure - it's a little more work, but it's the right thing to do. I think we
> should actually break this into 3 steps:
>
> 1. Introduce the new Flags struct and have the existing
> BinaryWithFlagsSDNode use them only for nsw/nuw/exact (NFC).
> 2. Convert BinaryWithFlagsSDNode to the broader SDNodeWithFlags class and
> extend the potential users to the FP nodes (NFC).
> 3. Update DAGCombiner to use the 'arcp' flag and add the new test cases.
>
> First patch coming up...
>
>
> http://reviews.llvm.org/D8900
>
> EMAIL PREFERENCES
>   http://reviews.llvm.org/settings/panel/emailpreferences/
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150428/83f5e3a8/attachment.html>