<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Aug 10, 2020 at 11:24 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">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></blockquote><div><br></div><div>Oh right. Do you have any suggestions on how to improve the wording?</div><div><br></div><div>Cheers,</div><div>Florian </div></div><div dir="ltr" class="gmail_signature"><br></div></div>