rsmith added inline comments.

================
Comment at: lib/Serialization/ASTReader.cpp:2554-2556
+      Module *M = PP.getHeaderSearchInfo().lookupModule(
+          F.ModuleName,
+          /*AllowSearch*/ true, /*SearchTopLevelOnly*/ true);
----------------
This is only trying to catch the case where we've already loaded a module map 
for the module and the two conflict. We should just set `AllowSearch` to 
`false` here. If we want to also perform validation on the case where we load 
the module map after importing another definition, that should be handled 
if/when we load the module map.


https://reviews.llvm.org/D32111



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to