[PATCH] D62187: Delete default constructors, copy constructors, move constructors, copy assignment, move assignment operators on Expr, Stmt and Decl

[Dmitri Gribenko via Phabricator via cfe-commits]

gribozavr marked 2 inline comments as done.
gribozavr added inline comments.


================
Comment at: clang/include/clang/AST/Stmt.h:1057
 
+  Stmt() = delete;
   Stmt(const Stmt &) = delete;
----------------
ilya-biryukov wrote:
> gribozavr wrote:
> > ilya-biryukov wrote:
> > > NIT: Move the deleted declarations to the start of the class?
> > > Or move the deleted declarations in other classes to the first public section?
> > > 
> > > (For consistency)
> > I put these new declarations close to the "primary" constructor.
> Why not move declarations inside `Stmt` to the primary constructor too?
Done.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D62187/new/

https://reviews.llvm.org/D62187