rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land.
Looks good to me. Please first mail cfe-dev announcing this change and wait a day or so for anyone using this feature to speak up before committing. ================ Comment at: test/Coverage/ast-printing.c:7 -// RUN: %clang_cc1 -ast-dump-all %s -// RUN: %clang_cc1 -print-decl-contexts %s - ---------------- Please reinstate this file and only remove the one `RUN:` line that tests `-print-decl-contexts`. ================ Comment at: test/Coverage/ast-printing.cpp:7 -// RUN: %clang_cc1 -std=c++14 -ast-dump-all %s -// RUN: %clang_cc1 -std=c++14 -print-decl-contexts %s -// RUN: %clang_cc1 -std=c++14 -fdump-record-layouts %s ---------------- Likewise here. Repository: rC Clang https://reviews.llvm.org/D52529 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
