[PATCH] Enable layering check in unavailable modules

Manuel Klimek klimek at google.com
Fri Dec 20 02:42:09 PST 2013


  lg


================
Comment at: include/clang/Lex/ModuleMap.h:236
@@ +235,3 @@
+  ///
+  /// \param File The included file.
+  void diagnoseHeaderInclusion(Module *RequestingModule,
----------------
I'd probably remove all the empty comment lines...

================
Comment at: lib/Lex/ModuleMap.cpp:205
@@ +204,3 @@
+                                        const FileEntry *File) {
+  if (RequestingModule != SourceModule)
+    return;
----------------
Please re-add the comment from the other file.


http://llvm-reviews.chandlerc.com/D2442



More information about the cfe-commits mailing list