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

Andrzej Warzynski via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 27 12:48:09 PDT 2022


awarzynski accepted this revision.
awarzynski added a comment.

LGTM, thanks for implementing this!



================
Comment at: clang/lib/Driver/ToolChains/Flang.cpp:98-99
+    } else
+      // Clang's "fast-honor-pragmas" option is not supported because it is
+      // non-standard and pragmas are not relevant to Fortran.
+      D.Diag(diag::err_drv_unsupported_option_argument)
----------------
[nit] "... and pragmas are not relevant to Fortran." Fortran has directives rather than pragmas. So it's not quite like pragmas are not supported. It's just that there's different mechanism instead.


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

https://reviews.llvm.org/D136080



More information about the cfe-commits mailing list