[PATCH] D94682: [clang][cli] Parse Lang and CodeGen options separately
Jan Svoboda via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Jan 14 09:02:59 PST 2021
jansvoboda11 added inline comments.
================
Comment at: clang/include/clang/Driver/Options.td:1831
+ MarshallingInfoString<LangOpts<"LaxVectorConversions">,
+ !strconcat(open_cl.KeyPath, " ? LangOptions::LaxVectorConversionKind::None"
+ " : LangOptions::LaxVectorConversionKind::All")>,
----------------
This replaces the conditional in `CompilerInvocation::setLangDefaults`.
The enum cases need to be fully qualified now, because we can't use `NormalizedValuesScope<"LangOptions::LaxVectorConversionKind">` due to the "arbitrary" expression for default value.
================
Comment at: clang/test/Frontend/diagnostics-order.c:12
// CHECK-NEXT: note: use {{.*}} for {{.*}} standard
+// CHECK: warning: optimization level '-O999' is not supported
----------------
The order of these diagnostics depends on the order of argument parsing. Because we've moved `CodeGenOpts` parsing after `LangOpts`, this needs to be adjusted.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D94682/new/
https://reviews.llvm.org/D94682
More information about the cfe-commits
mailing list