awarzynski added a comment.
All in all LGTM, but I'm not sure whether Flang should be defaulting to OpenMP
5.0. AFAIK, that's not supported yet.
================
Comment at: clang/lib/Driver/ToolChains/Flang.cpp:34
+ Args.AddAllArgs(CmdArgs, {options::OPT_ffixed_form,
+ options::OPT_ffree_form,
+ options::OPT_ffixed_line_length_EQ,
----------------
Is this clang-format? Looks like a big block of unrelated changes (not against
it).
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D150354/new/
https://reviews.llvm.org/D150354
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
