[PATCH] D60123: [AST] Forbid copy/move of Stmt.

2019-04-02 Thread Bruno Ricci via Phabricator via cfe-commits
riccibruno added a comment. In D60123#1451303 , @lebedev.ri wrote: > Probably `Decl` too? Yep. I though it was already done but after checking it seems I remembered incorrectly. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.

[PATCH] D60123: [AST] Forbid copy/move of Stmt.

2019-04-02 Thread Roman Lebedev via Phabricator via cfe-commits
lebedev.ri added a comment. Probably `Decl` too? Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60123/new/ https://reviews.llvm.org/D60123 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org

[PATCH] D60123: [AST] Forbid copy/move of Stmt.

2019-04-02 Thread Bruno Ricci via Phabricator via cfe-commits
riccibruno created this revision. riccibruno added a reviewer: aaron.ballman. riccibruno added a project: clang. Herald added a subscriber: cfe-commits. Statements and expressions are not supposed to be moved, and trying to do so is only going to result in tears. Someone tripped on this a few day