[PATCH] Hide pragma handler classes in ParsePragma.cpp

[Reid Kleckner]

Should have been fixed by r201825.


On Thu, Feb 20, 2014 at 3:54 PM, Juergen Ributzka <juergen at apple.com> wrote:

> On of your recent pragma changes broke our internal build.
>
> ******************** TEST 'Clang Tools :: pp-trace/pp-trace-pragma-ms.cpp'
> FAILED ********************
> Script:
> --
> pp-trace -ignore FileChanged,MacroDefined
> src/tools/clang/tools/extra/test/pp-trace/pp-trace-pragma-ms.cpp -target
> x86_64 -fms-extensions -w | FileCheck --strict-whitespace
> src/tools/clang/tools/extra/test/pp-trace/pp-trace-pragma-ms.cpp
> --
> Exit Code: 1
>
> Command Output (stderr):
> --
> src/tools/clang/tools/extra/test/pp-trace/pp-trace-pragma-ms.cpp:23:16:
> error: expected string not found in input
> // CHECK-NEXT: - Callback: PragmaComment
>                ^
> <stdin>:5:1: note: scanning from here
> - Callback: PragmaDirective
> ^
>
> Could you please take a look?
>
> Thanks
>
> Cheers,
> Juergen
>
> On Feb 20, 2014, at 2:59 PM, Reid Kleckner <rnk at google.com> wrote:
>
> >
> >  Closed by commit rL201820 (authored by @rnk).
> >
> > CHANGED PRIOR TO COMMIT
> >  http://llvm-reviews.chandlerc.com/D2834?vs=7219&id=7260#toc
> >
> > http://llvm-reviews.chandlerc.com/D2834
> >
> > COMMIT
> >  http://llvm-reviews.chandlerc.com/rL201820
> > _______________________________________________
> > cfe-commits mailing list
> > cfe-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140220/1edab645/attachment.html>