diff mbox series

[wwwdocs] gcc-12/changes.html: OpenMP (depobj/mutexinoutset for depend), OpenACC (-Wopenacc-parallelism)

Message ID 842c1337-a117-f260-bbb4-4e7f5a03546f@codesourcery.com
State New
Headers show
Series [wwwdocs] gcc-12/changes.html: OpenMP (depobj/mutexinoutset for depend), OpenACC (-Wopenacc-parallelism) | expand

Commit Message

Tobias Burnus April 26, 2021, 4:16 p.m. UTC
Hi all,

this patch documents a new OpenMP feature (many more to be added) in GCC 12.
And it documents a new flag for OpenACC.

Comments? Wording suggestions? I think for OpenMP, the sentence will be
modified several times before the release :-)

Tobias

-----------------
Mentor Graphics (Deutschland) GmbH, Arnulfstrasse 201, 80634 München Registergericht München HRB 106955, Geschäftsführer: Thomas Heurung, Frank Thürauf

Comments

Gerald Pfeifer April 26, 2021, 7:04 p.m. UTC | #1
On Mon, 26 Apr 2021, Tobias Burnus wrote:
> Comments? Wording suggestions? I think for OpenMP, the sentence will be 
> modified several times before the release :-)

Can I take this as a promise? :-)

+<ul>
+  <li>For Fortran, OpenMP 5.0 support has been extended for following features
+      which were before only available in C and C++:  <code>depobj</code>
+      and <code>mutexinoutset</code> can now also be used with the
+      <code>depend</code> clause.
+  </li>

I've been staring at this multiple times thinking on how to make it a
bit stronger/clearer.

How do you feel about

   "OpenMP 5.0 support for Fortran ... by the following features 
   which were available in C and C++ before:"

Or
 
  "Fortran gained support for the following OpenMP 5.0 features..."

?

(In marketing, which this kinds is, I suggest not saying "only" since
that risks diminishing the value we've had all along. Rather it's "even 
more/better now". ;-)

+  <li>The new warning flag <code>-Wopenacc-parallelism</code> was added for
+      OpenACC, which warns about potentially suboptimal choices related to
+      OpenACC parallelism.
+  </li>

Maybe break this as "... It warns about"?  That feels a bit more natural
to me than the "which" referring to an earlier term, though that may very
well be my subjective preference. ;-)


Please consider the above, and then it's okay, however you want to go
about it.

Gerald
Tobias Burnus April 27, 2021, 10:15 a.m. UTC | #2
Hi Gerald, hi all,

On 26.04.21 21:04, Gerald Pfeifer wrote:
> On Mon, 26 Apr 2021, Tobias Burnus wrote:
>> I think for OpenMP, the sentence will be
>> modified several times before the release :-)
> Can I take this as a promise? :-)

Given that Stage1 has just begun and several patches are
either waiting for review or to be finished and submitted: Yes :-)

> How do you feel about
>     "OpenMP 5.0 support for Fortran ... by the following features
>     which were available in C and C++ before:"
> Or
>    "Fortran gained support for the following OpenMP 5.0 features..."
I went for the former: it reduces the word smithing for non-Fortran-only
changes.

Thanks for the suggestions.

Tobias

PS: When looking at https://gcc.gnu.org/mirrors.html, I wondered whether
3 of 5 http mirrors links could move to https and given the trend that
FTP is phased out (on the server side and in the webbrowsers), whether
those ftp:// mirrors which also offer https:// should list https in
addition (or instead?).

-----------------
Mentor Graphics (Deutschland) GmbH, Arnulfstrasse 201, 80634 München Registergericht München HRB 106955, Geschäftsführer: Thomas Heurung, Frank Thürauf
Gerald Pfeifer April 27, 2021, 10:36 a.m. UTC | #3
On Tue, 27 Apr 2021, Tobias Burnus wrote:
> Thanks for the suggestions.

You're always welcome!

> PS: When looking at https://gcc.gnu.org/mirrors.html, I wondered whether
> 3 of 5 http mirrors links could move to https and given the trend that
> FTP is phased out (on the server side and in the webbrowsers), whether
> those ftp:// mirrors which also offer https:// should list https in
> addition (or instead?).

That's a good observation and recommendation; thank you!

I started to work on this; happy for you (or others) to join me over
the coming days. :)

Gerald
diff mbox series

Patch

diff --git a/htdocs/gcc-12/changes.html b/htdocs/gcc-12/changes.html
index 8f257e87..1022b5eb 100644
--- a/htdocs/gcc-12/changes.html
+++ b/htdocs/gcc-12/changes.html
@@ -40,6 +40,18 @@  a work-in-progress.</p>
 <!-- .................................................................. -->
 <h2 id="languages">New Languages and Language specific improvements</h2>
 
+<ul>
+  <li>For Fortran, OpenMP 5.0 support has been extended for following features
+      which were before only available in C and C++:  <code>depobj</code>
+      and <code>mutexinoutset</code> can now also be used with the
+      <code>depend</code> clause.
+  </li>
+  <li>The new warning flag <code>-Wopenacc-parallelism</code> was added for
+      OpenACC, which warns about potentially suboptimal choices related to
+      OpenACC parallelism.
+  </li>
+</ul>
+
 <!-- <h3 id="ada">Ada</h3> -->
 
 <!-- <h3 id="brig">BRIG (HSAIL)</h3> -->