Sorry it took me so long to get to this review. I agree with Sean regarding how the headers should be located relative to the header-list file. Using clang::HeaderSearch is probably overkill.
From: [email protected] [mailto:[email protected]] On Behalf Of Sean Silva Sent: Thursday, March 21, 2013 6:30 PM To: Thompson, John Cc: [email protected] Subject: Re: modularize tests - please review On Wed, Mar 20, 2013 at 9:26 PM, Thompson, John <[email protected]<mailto:[email protected]>> wrote: Sean, I tried using -I to fix the path problem, but modularize doesn't use the include path when trying to read the files from the source input list. That could be changed, but it's probably better to let it remain explicit where the sources come from. Perhaps I could change modularize to look for the files relative to the list file, or add a separate option for specifying the directory to use for a reference, or perhaps both. I think it makes sense to look for the files relative to the list file, with a commandline option to override that behavior. -- Sean Silva
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
