[PATCH] D39304: [IR] redefine 'reassoc' fast-math-flag and add 'trans' fast-math-flag

[Michael Berg via llvm-commits]

With afn and contract not being recognized as part of fast, it was in both areas in that isFast was false under these circumstances.
Our solution has it in hand for us though with fast restored in the auto upgrader path.

Regards,
Michael

> On Feb 8, 2018, at 7:01 AM, Sanjay Patel via Phabricator <reviews at reviews.llvm.org> wrote:
> 
> spatel added a comment.
> 
> In https://reviews.llvm.org/D39304#1001295, @mcberg2017 wrote:
> 
>> As an llvm vendor for Apple targets we have actually run into the incompatibility issues with old fast and current fast, we elected to use version markers in the IR for in house code to tell us that the old fast is relevant in the bit code reader during auto upgrade for IR into new Fast format.
> 
> 
> Thanks for letting me know. Did this manifest as a performance problem only or was there a visible functional difference too?
> 
> 
> Repository:
>  rL LLVM
> 
> https://reviews.llvm.org/D39304
> 
> 
>