[PATCH] D80315: Fix CC1 command line options mapping into fast-math flags.

Wed May 27 18:02:23 PDT 2020

rjmccall added a comment.

In D80315#2058735 <https://reviews.llvm.org/D80315#2058735>, @michele.scandale wrote:

> In D80315#2058549 <https://reviews.llvm.org/D80315#2058549>, @rjmccall wrote:
>
> > The code cleanups all seems reasonable.  The actual changes in code-generation changes are because we were failing to set these reliably?
>
>
> Most of them yes.
>
> In the CUDA test we there is now `contract` because we honor the default contraction mode that for CUDA is set to fast.

Right, and we weren't honoring that mode before?

> In `clang/test/CodeGen/complex-math.c` I've added `-ffp-contract=fast` to the command line option because `-ffast-math` at the CC1 level does not change the default contraction mode.

Oh, I see, makes sense.  So there was inconsistent treatment of the options, where one thing observed it but others didn't, and that's been fixed now.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80315