[PATCH] D50913: [FPEnv] Don't need copysign/fabs/fneg constrained intrinsics

Kevin P. Neal via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 24 10:38:21 PDT 2018


kpn added a comment.

In https://reviews.llvm.org/D50913#1212622, @cameron.mcinally wrote:

> > Did we decide that fast math flags can't be applied in the presence of constrained operations?
>
> I haven't thought about this before today, but some of the flags we would want. E.g. reassoc/contract/afn.
>
> In a pure trap-safe mode, contract/afn should be turned off. But, generally speaking, our users would prefer to see optimized code that misses some right-at-the-edge cases.


Over here we'd want contract and afn but not reassoc.


Repository:
  rL LLVM

https://reviews.llvm.org/D50913





More information about the llvm-commits mailing list