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

[Ben Langmuir]

So would the change you’re envisioning be to subclass VerifyJobAction and/or add a new ActionClass?  All of the other ActionClasses are unambiguous in what tool they need.

Ben

On Feb 5, 2014, at 2:35 PM, Ben Langmuir <blangmuir at apple.com> wrote:

> Sounds reasonable, I’ll look into it.
> 
> Ben
> 
> On Feb 5, 2014, at 2:33 PM, Eric Christopher <echristo at gmail.com> wrote:
> 
>> The idea was that the hook could be used for any verification, just a specific one of debug at the moment. It can easily be changed to support more verifications (and should). :)
>> 
>> -eric
>> 
>> On Feb 5, 2014 2:32 PM, "Ben Langmuir" <blangmuir at apple.com> wrote:
>> 
>> On Feb 5, 2014, at 11:00 AM, Eric Christopher <echristo at gmail.com> wrote:
>> 
>> > Out of curiosity would it work to have this as a Verify action rather
>> > than a Compile action that has no output?
>> >
>> > -eric
>> 
>> VerifyJobAction seems to be specific to verifying debug information.  Perhaps that should change?  Or maybe it should be renamed to VerifyDebugInfoJobAction?  I’m not sure.
>> 
>> Ben
> 
> _______________________________________________
> 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/20140205/237a27c0/attachment.html>