[PATCH] Add a -verify-pch option to clang -cc1

Eric Christopher echristo at gmail.com
Wed Feb 5 11:00:54 PST 2014


Out of curiosity would it work to have this as a Verify action rather
than a Compile action that has no output?

-eric

On Fri, Jan 24, 2014 at 9:58 AM, Ben Langmuir <blangmuir at apple.com> wrote:
> Based on an off-list discussion, I've updated this patch to not check for configuration mismatches (e.g. different langopts), since checking for staleness is the interesting case to check for and adding all the requisite langopts to the command line increases the complexity of using the feature.
>
> Ben
>
>
>
>
> On Jan 21, 2014, at 8:18 PM, Ben Langmuir <blangmuir at apple.com> wrote:
>
>>
>>> Looks good!
>>> Could you also add a driver option (maybe "--verify-pch") ?
>>
>> Sure, I've added -verify-pch to the driver as well.  I kept the same spelling between the two options.  The biggest change here is that we don't want the driver to pass through -x TYPE to cc1 for an input file that is a pch when we are verifying, since we want to verify the *source* language.  Right now I am omitting "-x TYPE" when -verify-pch is set AND the input file has a .pch extension.  I'm not sure whether checking the extension is a good idea, but it is consistent with other bits of the compiler that will assume the input is an object file if it has no extension and no explicit -x option.  I'm happy to change it.
>>
>> Ben
>>
>> <verify-pch.patch>
>>
>>
>> _______________________________________________
>> cfe-commits mailing list
>> cfe-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>
> _______________________________________________
> 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