[PATCH] Enable layering check in unavailable modules

Dmitri Gribenko gribozavr at gmail.com
Thu Dec 19 11:31:03 PST 2013



================
Comment at: lib/Lex/ModuleMap.cpp:165-166
@@ -166,8 +164,4 @@
   HeadersMap::iterator Known = Headers.find(File);
-
-  // If we've found a builtin header within Clang's builtin include directory,
-  // load all of the module maps to see if it will get associated with a
-  // specific module (e.g., in /usr/include).
   if (Known == Headers.end() && File->getDir() == BuiltinIncludeDir &&
       isBuiltinHeader(llvm::sys::path::filename(File->getName()))) {
     HeaderInfo.loadTopLevelSystemModules();
----------------
Why do we need the isBuiltinHeader() check?  It returns false for some builtin headers, for example, xmmintrin.h.



http://llvm-reviews.chandlerc.com/D2442



More information about the cfe-commits mailing list