[cfe-commits] r83594 - in /cfe/trunk: include/clang/Frontend/InitHeaderSearch.h lib/Frontend/InitHeaderSearch.cpp tools/clang-cc/clang-cc.cpp

[Chris Lattner]

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