[PATCH] Hide pragma handler classes in ParsePragma.cpp

[Juergen Ributzka]

Yup, that fixed it. Thanks

On Feb 20, 2014, at 4:10 PM, Juergen Ributzka <juergen at apple.com> wrote:

> Clang commits is a little slow today. It hasn’t shown up in my inbox yet, but I see that it is currently compiling.
> 
> On Feb 20, 2014, at 4:07 PM, Reid Kleckner <rnk at google.com> wrote:
> 
>> 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/f3035461/attachment.html>