[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