r265634 - Don't use PATH_MAX.

Sean Silva via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 6 18:58:15 PDT 2016


Author: silvas
Date: Wed Apr  6 20:58:14 2016
New Revision: 265634

URL: http://llvm.org/viewvc/llvm-project?rev=265634&view=rev
Log:
Don't use PATH_MAX.

This is a SmallVector anyway, and so the exact size doesn't matter.

clang\lib\Frontend\ModuleDependencyCollector.cpp(83) : error C2065: 'PATH_MAX' : undeclared identifier
clang\lib\Frontend\ModuleDependencyCollector.cpp(83) : error C2975: 'InternalLen' : invalid template argument for 'llvm::SmallString', expected compile-time constant expression
        llvm\include\llvm/ADT/SmallString.h(24) : see declaration of 'InternalLen'

Modified:
    cfe/trunk/lib/Frontend/ModuleDependencyCollector.cpp

Modified: cfe/trunk/lib/Frontend/ModuleDependencyCollector.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Frontend/ModuleDependencyCollector.cpp?rev=265634&r1=265633&r2=265634&view=diff
==============================================================================
--- cfe/trunk/lib/Frontend/ModuleDependencyCollector.cpp (original)
+++ cfe/trunk/lib/Frontend/ModuleDependencyCollector.cpp Wed Apr  6 20:58:14 2016
@@ -81,7 +81,7 @@ void ModuleDependencyCollector::attachTo
 }
 
 static bool isCaseSensitivePath(StringRef Path) {
-  SmallString<PATH_MAX> TmpDest = Path, UpperDest, RealDest;
+  SmallString<256> TmpDest = Path, UpperDest, RealDest;
   // Remove component traversals, links, etc.
   if (!real_path(Path, TmpDest))
     return true; // Current default value in vfs.yaml




More information about the cfe-commits mailing list