[clang] [clang][modules] Non-modular #import Should Respect Submodule Visibility (PR #170215)

[Qiongsi Wu via cfe-commits]

================
@@ -2373,10 +2386,8 @@ HeaderFileInfoTrait::ReadData(internal_key_ref key, const unsigned char *d,
     // implicit module import.
     SubmoduleID GlobalSMID = Reader.getGlobalSubmoduleID(M, LocalSMID);
     Module *Mod = Reader.getSubmodule(GlobalSMID);
-    ModuleMap &ModMap =
-        Reader.getPreprocessor().getHeaderSearchInfo().getModuleMap();
 
-    if (FE || (FE = getFile(key))) {
+    if (FE) {
----------------
qiongsiwu wrote:

Comments added! 

https://github.com/llvm/llvm-project/pull/170215