[cfe-commits] [PATCH] Fix for syntax checking a header file

Chandler Carruth chandlerc at google.com
Mon Jul 30 17:31:35 PDT 2012


Why different code from the phases::Compile code right below that handles
syntax only? I would expect these to be essentially identical.

Your test doesn't need '-OPTIONS1' or any suffix -- you only have one
compilation.

Also, make sure your subversion client submits that new test file with
native line endings rather than hard coded MS line endings.


On Mon, Jul 30, 2012 at 5:00 PM, Aaron Ballman <aaron at aaronballman.com>wrote:

> This patch addresses PR13343, where calling the driver with
> -fsyntax-only on a header file was creating a PCH file and ignoring
> the syntax check.  The fix was to special case a precompile action
> with fsyntax-check and set it to have no output file type (instead of
> a pch or pth file type).
>
> ~Aaron
>
> _______________________________________________
> 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/20120730/cc10cfb8/attachment.html>


More information about the cfe-commits mailing list