<div dir="ltr"><div dir="ltr">Excellent, thanks. I finally got around to pushing a patch adjusting the wording: <span style="color:rgb(0,0,0);font-family:Helvetica;font-size:12px">0814fcb7278a</span></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Aug 12, 2020 at 7:35 PM David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">Probably "Enable Clang Header Modules during compilation" or "Compile<br>
with Clang Header Modules"? in either case, make the "Clang Header<br>
Modules" link to <a href="https://clang.llvm.org/docs/Modules.html" rel="noreferrer" target="_blank">https://clang.llvm.org/docs/Modules.html</a> ?<br>
<br>
On Wed, Aug 12, 2020 at 8:05 AM Florian Hahn <<a href="mailto:flo@fhahn.com" target="_blank">flo@fhahn.com</a>> wrote:<br>
><br>
><br>
><br>
> On Mon, Aug 10, 2020 at 11:24 PM David Blaikie <<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>> wrote:<br>
>><br>
>> On Tue, Aug 4, 2020 at 9:02 AM Florian Hahn via llvm-commits<br>
>> <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br>
>> ><br>
>> ><br>
>> > Author: Florian Hahn<br>
>> > Date: 2020-08-04T16:59:39+01:00<br>
>> > New Revision: 05aa29efd7ecaea9631a3ac5c471a4a4749463a2<br>
>> ><br>
>> > URL: <a href="https://github.com/llvm/llvm-project/commit/05aa29efd7ecaea9631a3ac5c471a4a4749463a2" rel="noreferrer" target="_blank">https://github.com/llvm/llvm-project/commit/05aa29efd7ecaea9631a3ac5c471a4a4749463a2</a><br>
>> > DIFF: <a href="https://github.com/llvm/llvm-project/commit/05aa29efd7ecaea9631a3ac5c471a4a4749463a2.diff" rel="noreferrer" target="_blank">https://github.com/llvm/llvm-project/commit/05aa29efd7ecaea9631a3ac5c471a4a4749463a2.diff</a><br>
>> ><br>
>> > LOG: [docs] Mention LLVM_ENABLE_MODULES.<br>
>> ><br>
>> > Added:<br>
>> ><br>
>> ><br>
>> > Modified:<br>
>> >     llvm/docs/CMake.rst<br>
>> ><br>
>> > Removed:<br>
>> ><br>
>> ><br>
>> ><br>
>> > ################################################################################<br>
>> > diff  --git a/llvm/docs/CMake.rst b/llvm/docs/CMake.rst<br>
>> > index f96e34f21e80..da13726e8ca0 100644<br>
>> > --- a/llvm/docs/CMake.rst<br>
>> > +++ b/llvm/docs/CMake.rst<br>
>> > @@ -306,6 +306,9 @@ LLVM-specific variables<br>
>> >    scenario where a manual override may be desirable is when using Visual Studio<br>
>> >    2017's CMake integration, which would not be detected as an IDE otherwise.<br>
>> ><br>
>> > +**LLVM_ENABLE_MODULES**<br>
>> > +  Compile with C++ modules enabled.<br>
>><br>
>> This might be ambiguous/confusing. I believe this enables what's<br>
>> sometimes known as "Clang Header Modules" (might be worth a link to<br>
>> some doc explaining what it is) - not C++20 standard modules.<br>
><br>
><br>
> Oh right. Do you have any suggestions on how to improve the wording?<br>
><br>
> Cheers,<br>
> Florian<br>
><br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><br><a href="http://www.fhahn.com/" target="_blank">http://www.fhahn.com/</a><br></div>