[PATCH] D11844: [Modules] More descriptive diagnostics for misplaced import directive

Sean Silva via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 21 14:15:46 PDT 2015


silvas added inline comments.

================
Comment at: test/Modules/auto-module-import.m:89
@@ +88,3 @@
+			       expected-error {{import of module 'NoUmbrella.A' appears within function 'includeNotAtTopLevel'}} \
+			       expected-note {{consider marking the module as textual}}
+}
----------------
I think what this means is marking the *header* as textual. So I would suggest the wording "consider marking the header as textual".


http://reviews.llvm.org/D11844





More information about the cfe-commits mailing list