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 > wrote: On 10/03/15 14:21, Ben Langmuir wrote: Ack, yo

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 wrote: > > On 10/03/15 14:21, Ben Langmuir wrote: >> Ack, your response somehow got lost. >> >> LGTM. > Thanks! >> >>> +More information could be found

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 `_ 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 Ben Langmuir
Ack, your response somehow got lost. LGTM. > +More information could be found `here > `_ This should probably end with a period. > On Mar 10, 2015, at 1:10 AM, Vassil Vassilev wrote: > > ping... I really want to close that annoying bugzilla ticket ;)

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 wrote: On 09/17/2014 09:51 PM, Ben Langmuir wrote: Index: docs/LanguageExtensions.rst

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 wrote: On 09/17/2014 09:51 PM, Ben Langmuir wrote: Index: docs/LanguageExtensions.rst === --- docs/LanguageExtensions.rst (revision 217389) +++ d

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

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

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 ``__has_feature(cxx_rtti)`

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++ R

[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 (revis