[PATCH] D106876: Remove non-affecting module maps from PCM files.

Jan Svoboda via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 19 11:29:04 PDT 2022


jansvoboda11 added inline comments.


================
Comment at: clang/lib/Serialization/ASTWriter.cpp:1549
+    if (isModuleMap(File.getFileCharacteristic()) &&
+        !isSystem(File.getFileCharacteristic()) &&
+        !AffectingModuleMaps.empty() &&
----------------
Why do we never consider system module maps non-affecting? (i.e. always consider them affecting)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106876



More information about the cfe-commits mailing list