[PATCH] Hide pragma handler classes in ParsePragma.cpp

Juergen Ributzka juergen at apple.com
Thu Feb 20 15:54:03 PST 2014


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




More information about the cfe-commits mailing list