On Oct 8, 2009, at 4:29 PM, Mike Stump wrote:
> URL: http://llvm.org/viewvc/llvm-project?rev=83594&view=rev
> Log:
> Set up include paths for VC++ and Cygwin headers, along with the
> existing MinGW headers, plus the newer 4.4.0 version.  Patch by John
> Thompson.

Hi Mike,

I know this isn't your patch, but can you fix:

> void InitHeaderSearch::AddGnuCPlusPlusIncludePaths(std::string base,
> +                                                   std::string  
> arch) {

These to pass by const&, StringRef or something else appropriate?

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

Reply via email to