[PATCH] D78350: [AST] Build recovery expressions by default for C++.
Richard Smith - zygoloid via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Apr 28 23:57:38 PDT 2020
rsmith added inline comments.
================
Comment at: clang/include/clang/Basic/LangOptions.def:151
-COMPATIBLE_LANGOPT(RecoveryAST, 1, 0, "Preserve expressions in AST when encountering errors")
+COMPATIBLE_LANGOPT(RecoveryAST, 1, CPlusPlus, "Preserve expressions in AST when encountering errors")
----------------
Does this work? I would expect that we set all the options to the defaults at the same time, so this just sets this option to 0 (the default for `CPlusPlus`). If so, it'd be clearer to explicitly write that default here.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78350/new/
https://reviews.llvm.org/D78350
More information about the cfe-commits
mailing list