[PATCH] D120464: [clang][modules] Report module maps affecting `no_undeclared_includes` modules

Jan Svoboda via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 2 06:45:21 PST 2022


jansvoboda11 added inline comments.


================
Comment at: clang/test/Modules/add-remove-irrelevant-module-map.m:36-38
+#if __has_include("d.h") // This should use 'd.modulemap' in order to determine that 'd.h'
+                         // doesn't exist for 'c' because of its '[no_undeclared_includes]'.
+#endif
----------------
dexonsmith wrote:
> Would it be useful to also add a `#error` or similar inside the `#if`?
Yes, that makes sense. I'll add it before comitting.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120464



More information about the cfe-commits mailing list