On Fri, Feb 7, 2014 at 6:14 PM, Ben Langmuir <[email protected]> wrote:
> Hi Dmitri,
>
> +        // If we are reading a module, we will create a verification 
> timestamp,
> +        // so we verify all input files.  Otherwise, verify only user input
> +        // files.
> +        unsigned LastFileToValidate =
> +            F.Kind == MK_Module ? F.InputFilesLoaded.size() : Record[1];
> +        for (unsigned I = 0; I < LastFileToValidate; ++I) {
>             InputFile IF = getInputFile(F, I+1, Complain);
>             if (!IF.getFile() || IF.isOutOfDate())
>
>
> Should that be
>
> MK_Module && SkipVerifyIfValidatedAfter ?
>
> or is it intentional to stat the system files even if this new command line 
> option is missing?

Indeed, that was not intended.  Thanks!  Updated and rebased patch attached.

Dmitri

-- 
main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
(j){printf("%d\n",i);}}} /*Dmitri Gribenko <[email protected]>*/

Attachment: optimize-module-verification-v3.patch
Description: Binary data

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to