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

2015-03-10 Thread Ben Langmuir
Ack, your response somehow got lost. LGTM. +More information could be found `here http://clang.llvm.org/docs/Modules.html`_ This should probably end with a period. On Mar 10, 2015, at 1:10 AM, Vassil Vassilev vvasi...@cern.ch wrote: ping... I really want to close that annoying bugzilla

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

2015-03-10 Thread Vassil Vassilev
On 10/03/15 14:21, Ben Langmuir wrote: Ack, your response somehow got lost. LGTM. Thanks! +More information could be found `here http://clang.llvm.org/docs/Modules.html`_ This should probably end with a period. You are right, could you check it in with the period in the end? Vassil On

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

2015-03-10 Thread Vassil Vassilev
On 10/03/15 15:42, Ben Langmuir wrote: Committed as r231789. Feel free to close the PR, or I’ll do it later. Thanks for the patch! Thank you Ben! PR closed. Vassil On Mar 10, 2015, at 6:50 AM, Vassil Vassilev vvasi...@cern.ch mailto:vvasi...@cern.ch wrote: On 10/03/15 14:21, Ben Langmuir

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

2015-03-10 Thread Ben Langmuir
Committed as r231789. Feel free to close the PR, or I’ll do it later. Thanks for the patch! On Mar 10, 2015, at 6:50 AM, Vassil Vassilev vvasi...@cern.ch wrote: On 10/03/15 14:21, Ben Langmuir wrote: Ack, your response somehow got lost. LGTM. Thanks! +More information could be

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

2015-03-10 Thread Vassil Vassilev
ping... I really want to close that annoying bugzilla ticket ;) On 18/09/14 20:08, Vassil Vassilev wrote: On 18/09/14 18:04, Ben Langmuir wrote: On Sep 18, 2014, at 3:47 AM, Vassil Vassilev vasil.georgiev.vasi...@cern.ch wrote: On 09/17/2014 09:51 PM, Ben Langmuir wrote: Index:

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

2014-09-18 Thread Vassil Vassilev
On 09/17/2014 09:51 PM, Ben Langmuir wrote: Index: docs/LanguageExtensions.rst === --- docs/LanguageExtensions.rst (revision 217389) +++ docs/LanguageExtensions.rst (working copy) @@ -477,6 +477,13 @@ Use

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

2014-09-18 Thread Ben Langmuir
On Sep 18, 2014, at 3:47 AM, Vassil Vassilev vasil.georgiev.vasi...@cern.ch wrote: On 09/17/2014 09:51 PM, Ben Langmuir wrote: Index: docs/LanguageExtensions.rst === --- docs/LanguageExtensions.rst (revision 217389) +++

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

2014-09-18 Thread Vassil Vassilev
On 18/09/14 18:04, Ben Langmuir wrote: On Sep 18, 2014, at 3:47 AM, Vassil Vassilev vasil.georgiev.vasi...@cern.ch wrote: On 09/17/2014 09:51 PM, Ben Langmuir wrote: Index: docs/LanguageExtensions.rst === ---

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

2014-09-17 Thread Vassil Vassilev
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 Index: docs/LanguageExtensions.rst === --- docs/LanguageExtensions.rst

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

2014-09-17 Thread 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