[cfe-dev] fp-contract=fast and pragmas
John McCall via cfe-dev
cfe-dev at lists.llvm.org
Fri Jun 25 14:43:49 PDT 2021
On 25 Jun 2021, at 17:30, Kaylor, Andrew wrote:
> Hi everyone,
>
> I've been exploring clang's fp-contract behavior recently, and I see
> that when '-ffp-contract=fast' is used it can't be overridden with a
> pragma. I would have regarded this as a bug (and in fact, a bug has
> been filed https://bugs.llvm.org/show_bug.cgi?id=39679). However, I've
> found some discussions on the mailing lists that described this as
> expected behavior.
>
> Last October, Sam Liu added support for a new setting
> ('fp-contract=fast-honor-pragmas') and updated the clang documentation
> to reflect the behavior of fp-contract=fast. See
> https://reviews.llvm.org/D90174. I feel very strongly that this should
> have been done the other way around -- fp-contract=fast should honor
> pragmas and if we need an option that doesn't that could be added.
>
> In the above review, John McCall asked what "other compilers" do.
> Steve Canon showed that GCC doesn't honor the pragma. If I may humbly
> offer another "other compiler", ICC (which doesn't distinguish
> between 'on' and 'fast' for fp-contract) does respect the pragma
> (https://godbolt.org/z/x5r9WdYb4). I'm not saying that ICC should be
> treated as a reference implementation over GCC or anything like that,
> but I am saying that its behavior strikes me as more correct than what
> GCC or clang currently do.
>
> Thoughts and opinions?
I don’t disagree with you in the abstract, but we consider this a
GCC-designed feature. ICC’s value as contrary precedent appears
especially weak because, as you point out, they don’t really implement
`-ffp-contract=fast`.
There are plenty of other GCC-designed things that I don’t
particularly like the design of, but where we nonetheless consider
ourselves bound by their behavior.
John.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20210625/995e892a/attachment.html>
More information about the cfe-dev
mailing list