[PATCH] Add a new flag -fmodules-require-modular-includes

Richard Smith richard at metafoo.co.uk
Sun Apr 13 18:15:21 PDT 2014


Have you considered making this be a normal warning flag, instead of a
language options and an error?


On Thu, Apr 10, 2014 at 6:43 AM, Ben Langmuir <blangmuir at apple.com> wrote:

> As suggested off-list, this updated patch only affects frameworks for now,
> although the intent is to include all modules in the future.  It also has
> small fix for submodules including non-modular content and files that are
> nested inside umbrella directories.
>
> Ben
>
>
>
>
> On Apr 8, 2014, at 5:09 PM, Ben Langmuir <blangmuir at apple.com> wrote:
>
> > When set, the new flag enforces that all of the files included by a
> module are themselves part of a module, or are explicitly excluded by some
> module. This will not affect headers that are part of the module being
> built, nor files included outside of the module build (e.g. in an
> implementation file with -fmodule-name set).
> >
> > Ben
> >
> >
> <non-modular-includes.patch>_______________________________________________
> > cfe-commits mailing list
> > cfe-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140413/2e8b22c7/attachment.html>


More information about the cfe-commits mailing list