ABataev marked 2 inline comments as done. ================ Comment at: lib/AST/DeclPrinter.cpp:197 @@ -196,3 +196,3 @@ -void DeclPrinter::prettyPrintAttributes(Decl *D) { +void DeclPrinter::prettyPrintAttributes(Decl *D, bool PrintPragmas) { if (Policy.PolishForDeclaration) ---------------- aaron.ballman wrote: > Would it make more sense to add a prettyPrintPragmas(Decl *D) function > instead? Ok, will do
================ Comment at: test/SemaCXX/pragma-init_seg.cpp:1 @@ -1,2 +1,2 @@ -// RUN: %clang_cc1 -fsyntax-only -verify -fms-extensions %s -triple x86_64-pc-win32 +// RUN: %clang_cc1 -fsyntax-only -verify -fms-extensions %s -triple x86_64-pc-win32 -ast-print | FileCheck %s // RUN: %clang_cc1 -fsyntax-only -verify -fms-extensions %s -triple i386-apple-darwin13.3.0 ---------------- aaron.ballman wrote: > I think this should be a new test under Misc\. We have ast-print-pragmas.cpp > that looks like a good place for it to go. Ok http://reviews.llvm.org/D13546 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits