[PATCH] D134224: [clang][modules][deps] Report modulemaps describing excluded headers

[Michael Spencer via Phabricator via cfe-commits]

Bigcheese added a comment.

I think this is fine, but I need to think more about the ModuleDepCollector.cpp change.



================
Comment at: clang/lib/Lex/ModuleMap.cpp:1275
   // specified module?
-  (void) Headers[Header.Entry];
+  (void) Headers[Header.Entry].push_back(KH);
 
----------------
Cast to `void` no longer needed.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D134224/new/

https://reviews.llvm.org/D134224