Please add a test that tries to import a module that contains a missing file. 
I expect that'll assert in `Module::isAvailable`, because it can't explain why 
the module is unavailable.


================
Comment at: lib/Lex/ModuleMap.cpp:1569
@@ +1568,3 @@
+    // If we find a module that has a missing header, we mark this module as
+    // unavailable. Layering warnings like -fmodules-decluse can still be used.
+    ActiveModule->IsAvailable = false;
----------------
Presumably other submodules of the same module can still be built and used, 
too? (I don't think there's anything specific to layering warnings here?)


http://llvm-reviews.chandlerc.com/D2352
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to