[Patch][BugzillaID#18985] Document __has_feature(modules)

[Ben Langmuir]

> Index: docs/LanguageExtensions.rst
> ===================================================================
> --- docs/LanguageExtensions.rst	(revision 217389)
> +++ docs/LanguageExtensions.rst	(working copy)
> @@ -477,6 +477,13 @@
>  Use ``__has_feature(cxx_rtti)`` to determine if C++ RTTI has been enabled.  For
>  example, compiling code with ``-fno-rtti`` disables the use of RTTI.
>  
> +C++ Modules
> +^^^^^^^^
> +
> +Use ``__has_feature(modules)`` to determine if experimental C++ Modules have
> +been enabled. For example, compiling code with ``-fmodules`` enables the use of
> +C++ Modules.
> +
>  C++11
>  -----
> 

Why are we making this specific to C++ modules?  Modules are supported in C/ObjC. And to actually get modules in C++ you also need -fcxx-modules.

Ben

> On Sep 17, 2014, at 12:59 AM, Vassil Vassilev <vasil.georgiev.vasilev at cern.ch> wrote:
> 
> Hi,
>  I am attaching a patch addressing http://llvm.org/bugs/show_bug.cgi?id=18985
>  I wasn't sure whether I had to say 'experimental C++ modules'.
> Vassil
> <Bug18985.diff>_______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits