[PATCH] D26105: Allow CaseStmt to be initialized with a SubStmt

2017-10-25 Thread Gábor Horváth via Phabricator via cfe-commits
xazax.hun abandoned this revision. xazax.hun added a comment. Since r316069 this is no longer relevant. https://reviews.llvm.org/D26105 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commit

[PATCH] D26105: Allow CaseStmt to be initialized with a SubStmt

2016-10-31 Thread Alexey Bataev via cfe-commits
ABataev added a comment. Add a test Comment at: include/clang/AST/Stmt.h:697 + SourceLocation ellipsisLoc, SourceLocation colonLoc, + Stmt *SubStmt=nullptr) : SwitchCase(CaseStmtClass, caseLoc, colonLoc) { I don't think you need this ne

[PATCH] D26105: Allow CaseStmt to be initialized with a SubStmt

2016-10-28 Thread Kareem Khazem via cfe-commits
khazem updated this revision to Diff 76272. khazem added a comment. (added more context) https://reviews.llvm.org/D26105 Files: include/clang/AST/Stmt.h lib/AST/ASTImporter.cpp Index: lib/AST/ASTImporter.cpp === --- lib/AST/A

[PATCH] D26105: Allow CaseStmt to be initialized with a SubStmt

2016-10-28 Thread Kareem Khazem via cfe-commits
khazem created this revision. khazem added reviewers: a.sidorin, xazax.hun, doug.gregor, rsmith. khazem added subscribers: phosek, cfe-commits, smklein. The CaseStmt constructor now takes an optional SubStmt argument to initialize its SubExprs field with. This fixes an issue where the ASTImporter