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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to