[PATCH] D151834: Include math-errno with fast-math

[Aaron Ballman via Phabricator via cfe-commits]

aaron.ballman added a comment.

In D151834#4643925 <https://reviews.llvm.org/D151834#4643925>, @uabelho wrote:

> Hi @zahiraam ,
>
> I have a couple of downstream testcases that fail with this patch.
> Before
>
>   > clang bbi-86364.c -lm -O3
>   > ./a.out
>
> passed but with the patch the assert in the program fails:
>
>   a.out: bbi-86364.c:9: int main(): Assertion `(*__errno_location ()) == 33' failed.
>
> Is this as expected?
>
> F29200339: bbi-86364.c <https://reviews.llvm.org/F29200339>

This seems unexpected to me and it seems to relate to whether you include errno.h or not: https://godbolt.org/z/EPWzazx9r -- @zahiraam do you have ideas as to what's going on?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D151834/new/

https://reviews.llvm.org/D151834