Ping: [PATCH] wwwdocs: Add D front-end section for GCC 10 changes

Hi,

Updated the patch to include missed changes, and slighted reworded some entries
to make them clearer/read easier.

OK to commit?

Iain.

---
 htdocs/gcc-10/changes.html | 35 +++++++++++++++++++++++++++++++++++
 1 file changed, 35 insertions(+)

Ping.

On 07/05/2020 16:04, Iain Buclaw via Gcc-patches wrote:
> Hi,
> 
> Updated the patch to include missed changes, and slighted reworded some entries
> to make them clearer/read easier.
> 
> OK to commit?
> 
> Iain.
> 
> ---
>  htdocs/gcc-10/changes.html | 35 +++++++++++++++++++++++++++++++++++
>  1 file changed, 35 insertions(+)
> 
> diff --git a/htdocs/gcc-10/changes.html b/htdocs/gcc-10/changes.html
> index 41c2dc0d..f10cfd56 100644
> --- a/htdocs/gcc-10/changes.html
> +++ b/htdocs/gcc-10/changes.html
> @@ -432,6 +432,41 @@ a work-in-progress.</p>
>    </li>
>  </ul>
>  
> +<h3 id="d">D</h3>
> +<ul>
> +  <li>Support for <code>static foreach</code> has been implemented.</li>
> +  <li>Aliases can now be created directly from any <code>__trait</code> that
> +      return symbols or tuples.  Previously, an <code>AliasSeq</code> was
> +      necessary in order to alias their return.
> +  </li>
> +  <li>It is now possible to detect the language ABI specified for a struct,
> +      class, or interface using <code>__traits(getLinkage, ...)</code>
> +  <li>Support for <code>core.math.toPrec</code> intrinsics have been added.
> +      These intrinsics guarantee the rounding to specific floating-point
> +      precisions at required points in the code.
> +  </li>
> +  <li>Support for <code>pragma(inline)</code> has been implemented.  Previously
> +      the pragma was recognized, but had no effect on the compilation.
> +  </li>
> +  <li>Optional parentheses in <code>asm</code> operands are now deprecated and
> +      will be removed in a future release.
> +  </li>
> +  <li>All content imported files are now included in the make dependency list
> +      when compiling with <code>-M</code>.
> +  </li>
> +  <li>Compiler recognized attributes provided by the <code>gcc.attribute</code>
> +      module will now take effect when applied to function prototypes as well
> +      as when applied to full function declarations.
> +  </li>
> +  <li>Added <code>--enable-libphobos-checking</code> configure option to
> +      control whether run-time checks are compiled into the D runtime library.
> +  </li>
> +  <li>Added <code>--with-libphobos-druntime-only</code> configure option to
> +      allow specifying whether to build only the core D runtime library, or
> +      both the core and standard libraries into libphobos.
> +  </li>
> +</ul>
> +
>  <h3 id="fortran">Fortran</h3>
>  <ul>
>    <li><code>use_device_addr</code> of version 5.0 of the
>

On 07/05/2020 16:04, Iain Buclaw via Gcc-patches wrote:
> Hi,
> 
> Updated the patch to include missed changes, and slighted reworded some entries
> to make them clearer/read easier.
> 


I've gone ahead and pushed it, after someone else did a review of the wording.

Iain.

On Wed, 2020-05-13 at 15:00 +0200, Iain Buclaw via Gcc-patches wrote:
> Ping.
> 
> On 07/05/2020 16:04, Iain Buclaw via Gcc-patches wrote:
> > Hi,
> > 
> > Updated the patch to include missed changes, and slighted reworded some
> > entries
> > to make them clearer/read easier.
> > 
> > OK to commit?
OK.  In fact, this seems like you shouldn't need reviews -- you're just updating
the docs for D.

jeff
> >

On Thu, 14 May 2020, Jeff Law wrote:
>> OK to commit?
> OK.  In fact, this seems like you shouldn't need reviews -- you're just 
> updating the docs for D.

Agreed, though always happy to help and provide review and feedback
- sometimes just a bit slow as in this case.  But, indeed, you can
self approve, Iain.


Here is some specific feedback on the patch:
 
+  <li>Aliases can now be created directly from any <code>__trait</code> that
+      return symbols or tuples.  Previously, an <code>AliasSeq</code> was

"returns" ?

+  <li>It is now possible to detect the language ABI specified for a struct,
+      class, or interface using <code>__traits(getLinkage, ...)</code>

Full stop.

+  <li>Support for <code>core.math.toPrec</code> intrinsics have been added.

"has been"

+      These intrinsics guarantee the rounding to specific floating-point
+      precisions at required points in the code.

Is this about required points (by the standard) or specified (by the 
user/programmer)?

+  <li>Optional parentheses in <code>asm</code> operands are now deprecated and
+      will be removed in a future release.

Maybe omit "now" of which there's a number in the vicinity already?

+  <li>Added <code>--enable-libphobos-checking</code> configure option to

"Added a..."?

+  <li>Added <code>--with-libphobos-druntime-only</code> configure option to

Same here?

+      allow specifying whether to build only the core D runtime library, or
+      both the core and standard libraries into libphobos.

This reads a bit complicated.  Maybe "option to indicate whether to 
build..." ?


Please consider those items and questions and share the patch as you'll
install it - no need for review or anything. :)

Thanks,
Gerald

On 14/05/2020 17:04, Gerald Pfeifer wrote:
> On Thu, 14 May 2020, Jeff Law wrote:
>>> OK to commit?
>> OK.  In fact, this seems like you shouldn't need reviews -- you're just 
>> updating the docs for D.
> 
> Agreed, though always happy to help and provide review and feedback
> - sometimes just a bit slow as in this case.  But, indeed, you can
> self approve, Iain.
> 
> 
> Here is some specific feedback on the patch:
>  
> +  <li>Aliases can now be created directly from any <code>__trait</code> that
> +      return symbols or tuples.  Previously, an <code>AliasSeq</code> was
> 
> "returns" ?
> 

The "s" should instead be on:

"<code>__traits</code> that return symbols or tuples."

I don't *think* "returns" is necessary, there's already a plethora of plurals.
To be sure though, I put it through a grammar checker, and it flags up "returns"
as an error of subject-verb agreement in the sentence.

> +  <li>It is now possible to detect the language ABI specified for a struct,
> +      class, or interface using <code>__traits(getLinkage, ...)</code>
> 
> Full stop.
> 
> +  <li>Support for <code>core.math.toPrec</code> intrinsics have been added.
> 
> "has been"
> 
> +      These intrinsics guarantee the rounding to specific floating-point
> +      precisions at required points in the code.
> 
> Is this about required points (by the standard) or specified (by the 
> user/programmer)?
> 

By user/programmer, so I've changed it to "specified".

> +  <li>Optional parentheses in <code>asm</code> operands are now deprecated and
> +      will be removed in a future release.
> 
> Maybe omit "now" of which there's a number in the vicinity already?
> 
> +  <li>Added <code>--enable-libphobos-checking</code> configure option to
> 
> "Added a..."?
> 
> +  <li>Added <code>--with-libphobos-druntime-only</code> configure option to
> 
> Same here?
> 
> +      allow specifying whether to build only the core D runtime library, or
> +      both the core and standard libraries into libphobos.
> 
> This reads a bit complicated.  Maybe "option to indicate whether to 
> build..." ?
> 
> 
> Please consider those items and questions and share the patch as you'll
> install it - no need for review or anything. :)
> 


Installed.

diff --git a/htdocs/gcc-10/changes.html b/htdocs/gcc-10/changes.html
index 8eaa3326..b9fae2c6 100644
--- a/htdocs/gcc-10/changes.html
+++ b/htdocs/gcc-10/changes.html
@@ -497,6 +497,41 @@ int get_naïve_pi() {
   </li>
 </ul>
 
+<h3 id="d">D</h3>
+<ul>
+  <li>Support for <code>static foreach</code> has been implemented.</li>
+  <li>Aliases can now be created directly from any <code>__traits</code> that
+      return symbols or tuples.  Previously, an <code>AliasSeq</code> was
+      necessary in order to alias their return.
+  </li>
+  <li>It is now possible to detect the language ABI specified for a struct,
+      class, or interface using <code>__traits(getLinkage, ...)</code>.
+  <li>Support for <code>core.math.toPrec</code> intrinsics has been added.
+      These intrinsics guarantee the rounding to specific floating-point
+      precisions at specified points in the code.
+  </li>
+  <li>Support for <code>pragma(inline)</code> has been implemented.  Previously
+      the pragma was recognized, but had no effect on the compilation.
+  </li>
+  <li>Optional parentheses in <code>asm</code> operands are deprecated and will
+      be removed in a future release.
+  </li>
+  <li>All content imported files are now included in the make dependency list
+      when compiling with <code>-M</code>.
+  </li>
+  <li>Compiler recognized attributes provided by the <code>gcc.attribute</code>
+      module will now take effect when applied to function prototypes as well
+      as when applied to full function declarations.
+  </li>
+  <li>Added a <code>--enable-libphobos-checking</code> configure option to
+      control whether run-time checks are compiled into the D runtime library.
+  </li>
+  <li>Added a <code>--with-libphobos-druntime-only</code> configure option to
+      indicate whether to build only the core D runtime library, or both the
+      core and standard libraries into libphobos.
+  </li>
+</ul>
+
 <h3 id="fortran">Fortran</h3>
 <ul>
   <li><code>use_device_addr</code> of version 5.0 of the