[PATCH] propagate IR-level fast-math-flags to DAG nodes, disabled by default

Sanjay Patel spatel at rotateright.com
Wed Jul 29 11:51:17 PDT 2015


For reference, fixes for PR24141 are in:

http://reviews.llvm.org/rL243293http://reviews.llvm.org/rL243498http://reviews.llvm.org/rL243500

...assuming those don't cause new breakage, it's finally, really time to
flip the FMF flag...and wait for more breakage.

On Mon, Jul 20, 2015 at 11:34 AM, Sanjay Patel <spatel at rotateright.com>
wrote:

>
>
> On Fri, Jul 17, 2015 at 4:42 PM, Nick Lewycky <nlewycky at google.com> wrote:
>
>> On 9 July 2015 at 15:27, Sanjay Patel <spatel at rotateright.com> wrote:
>>
>>> Hi Nick,
>>>
>>> The reciprocal fix is in at r241826. Please let me know when you find
>>> another failure. Thanks!
>>>
>>
>> Thank you! I tested it and found exactly nothing wrong. Hopefully I
>> didn't goof the testing, but I think it's time to flip the flag to true,
>> then wait a few days before removing the flag entirely.
>>
>
> Wow...I'm surprised, but I'll hope for the best. Thanks for the testing!
>
>
> Note however, that we still have problems with PR24141 which is related to
>> unsafe fp math, but doesn't change with the -enable-fmf-dag flag.
>>
>
> Proposed fix:
> http://reviews.llvm.org/D11345
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150729/b57a9073/attachment.html>


More information about the llvm-commits mailing list