[PATCH] D136080: [flang] Add -ffp-contract option processing

[Slava Zakharin via Phabricator via cfe-commits]

vzakhari added inline comments.


================
Comment at: clang/lib/Driver/ToolChains/Flang.cpp:94
+          << Val << A->getOption().getName() << "fast";
+      FPContract = "fast";
+    } else
----------------
I know I suggested myself mapping `on` to `fast`, but it seems it will be more reasonable to map it to `off`.  Both my old classic flang and gfortran (12.2.0) do not generate FMAs under `on`.  Moreover, mapping it to `off` may allow us to adapt gcc's help message "if allowed by the language standard" (maybe not in this commit).


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

https://reviews.llvm.org/D136080