[PATCH] D26708: Fix -f[no-]reciprocal-math -ffast-math interaction, including LTO

David Majnemer via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 15 20:46:52 PST 2016


majnemer added a comment.

In https://reviews.llvm.org/D26708#596623, @wristow wrote:

> In https://reviews.llvm.org/D26708#596608, @majnemer wrote:
>
> > This change seems wrong.  Our IR defines `fast` as implying `arcp`: http://llvm.org/docs/LangRef.html#fast-math-flags
>
>
> So the change to make `-ffast-math -fno-reciprocal-math` work should be (a) remove the `fast` flag, and (b) add all the other ones (except `arcp`) to each relevant place?


This is the behavior I'd expect clang to have.


https://reviews.llvm.org/D26708





More information about the llvm-commits mailing list