[PATCH] D102251: Suppress Deferred Diagnostics in discarded statements.
Alexey Bataev via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue May 11 09:54:15 PDT 2021
ABataev added inline comments.
================
Comment at: clang/include/clang/AST/Stmt.h:2082-2083
/// Otherwise, or if the condition is value-dependent, returns None.
Optional<const Stmt*> getNondiscardedCase(const ASTContext &Ctx) const;
+ Optional<Stmt *> getNondiscardedCase(const ASTContext &Ctx);
----------------
Do you really need `Optional` here? Just nullptr is not enough?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D102251/new/
https://reviews.llvm.org/D102251
More information about the cfe-commits
mailing list