libgomp.texi: Add more to-be-implemented OpenMP 5.2 features / [wwwdocs] p.texi: Add more to-be-implemented OpenMP 5.2 features

Message ID 2a72bf67-8f28-66c0-3bb5-b52a5bd1bdd4@codesourcery.com
State New
Headers
Series libgomp.texi: Add more to-be-implemented OpenMP 5.2 features / [wwwdocs] p.texi: Add more to-be-implemented OpenMP 5.2 features |

Commit Message

Tobias Burnus May 25, 2022, 3:55 p.m. UTC
  I had a look at the OpenMP 5.2 changes and found some more,
which I think should be listed. Some were rather hidden, two
were only documented in the deprecate part.

I also added the 'begin declare target', which is a variant
to the delimited form of 'declare target' – the non-begin
variant is then deprecated in 5.1. (The begin variant permits
clauses, the other doesn't.)

OK? Comments?

Tobias

PS: For Fortran, 5.1 permitted with capture:
     [!$omp end atomic]
namely: the 'end atomic' is optional.
gfortran has implemented this – but 5.2 now made it required again,
treating it as bug. I am not sure whether this should now also be
listed as 'deleted' feature or not. Hmm.
-----------------
Siemens Electronic Design Automation GmbH; Anschrift: Arnulfstraße 201, 80634 München; Gesellschaft mit beschränkter Haftung; Geschäftsführer: Thomas Heurung, Frank Thürauf; Sitz der Gesellschaft: München; Registergericht München, HRB 106955
  

Comments

Tobias Burnus May 27, 2022, 7:41 a.m. UTC | #1
On 25.05.22 17:55, Tobias Burnus wrote:
> I had a look at the OpenMP 5.2 changes and found some more,
> which I think should be listed. Some were rather hidden, two
> were only documented in the deprecate part.

Fixes: Remove the last two items – they were already in the list (ups).

libgomp/libgomp.texi – forgot to update the default-map clause wording;
now again in sync with the html version.

> OK? Comments?
Tobias
-----------------
Siemens Electronic Design Automation GmbH; Anschrift: Arnulfstraße 201, 80634 München; Gesellschaft mit beschränkter Haftung; Geschäftsführer: Thomas Heurung, Frank Thürauf; Sitz der Gesellschaft: München; Registergericht München, HRB 106955
  
Jakub Jelinek May 27, 2022, 7:48 a.m. UTC | #2
On Fri, May 27, 2022 at 09:41:22AM +0200, Tobias Burnus wrote:
> On 25.05.22 17:55, Tobias Burnus wrote:
> > I had a look at the OpenMP 5.2 changes and found some more,
> > which I think should be listed. Some were rather hidden, two
> > were only documented in the deprecate part.
> 
> Fixes: Remove the last two items – they were already in the list (ups).
> 
> libgomp/libgomp.texi – forgot to update the default-map clause wording;
> now again in sync with the html version.

Ok.

	Jakub
  

Patch

gcc-13/changes.html - Update OpenMP / projects/gomp - add more to-do items

* gcc-13/changes.html (OpenMP): Add 'nowait' with 'taskwait'.
* projects/gomp/index.html: Add one 5.1 and several 5.2 additional
  to-be-implemented features.

diff --git a/htdocs/gcc-13/changes.html b/htdocs/gcc-13/changes.html
index c2610412..a262087f 100644
--- a/htdocs/gcc-13/changes.html
+++ b/htdocs/gcc-13/changes.html
@@ -40,7 +40,8 @@  a work-in-progress.</p>
   <ul>
     <li>The following OpenMP 5.1 features have been added: the
     <code>omp_all_memory</code> reserved locator, the <code>inoutset</code>
-    modifier to the <code>depend</code> clause and the
+    modifier to the <code>depend</code> clause, the <code>nowait</code>
+    clause for the <code>taskwait</code> directive and the
     <code>omp_target_is_accessible</code>, <code>omp_target_memcpy_async</code>,
     <code>omp_target_memcpy_rect_async</code> and
     <code>omp_get_mapped_ptr</code> API routines. Fortran now supports
diff --git a/htdocs/projects/gomp/index.html b/htdocs/projects/gomp/index.html
index 68531a57..f2b80ad8 100644
--- a/htdocs/projects/gomp/index.html
+++ b/htdocs/projects/gomp/index.html
@@ -630,6 +630,11 @@  than listed, depending on resolved corner cases and optimizations.</p>
     <td class="supported"><a href="../../gcc-13/changes.html#languages">GCC&nbsp;13</a></td>
     <td></td>
   </tr>
+  <tr>
+    <td><code>nowait</code> clause in <code>taskwait</code> directive</td>
+    <td class="supported"><a href="../../gcc-13/changes.html#languages">GCC&nbsp;13</a></td>
+    <td></td>
+  </tr>
   <tr>
     <td><em>target_device trait</em> in OpenMP Context</td>
     <td class="unsupported">No</td>
@@ -690,11 +695,6 @@  than listed, depending on resolved corner cases and optimizations.</p>
     <td class="unsupported">No</td>
     <td></td>
   </tr>
-  <tr>
-    <td><code>nowait</code> clause in <code>taskwait</code> directive</td>
-    <td class="unsupported">No</td>
-    <td></td>
-  </tr>
   <tr>
     <td><code>present</code> argument to <code>defaultmap</code> clause</td>
     <td class="unsupported">No</td>
@@ -725,6 +725,12 @@  than listed, depending on resolved corner cases and optimizations.</p>
     <td class="unsupported">No</td>
     <td></td>
   </tr>
+  <tr>
+    <td>Support <code>begin/end declare target</code> syntax in C/C++</td>
+    <td class="unsupported">No</td>
+    <td></td>
+  </tr>
+  <tr>
 </tbody>
 </table>
 
@@ -875,6 +881,36 @@  than listed, depending on resolved corner cases and optimizations.</p>
     <td class="unsupported">No</td>
     <td></td>
   </tr>
+  <tr>
+    <td>For Fortran, optional comma between directive and clause</td>
+    <td class="unsupported">No</td>
+    <td></td>
+  </tr>
+  <tr>
+    <td>Conforming device numbers and <code>omp_initial_device</code> and <code>omp_invalid_device</code> enum/PARAMETER</td>
+    <td class="unsupported">No</td>
+    <td></td>
+  </tr>
+  <tr>
+    <td><em>interop_types</em> in any position of the modifier list for the <code>init</code> clause of the <code>interop</code> construct<td>
+    <td class="unsupported">No</td>
+    <td></td>
+  </tr>
+  <tr>
+    <td>Initial value of <em>default-device-var</em> ICV with <code>OMP_TARGET_OFFLOAD=mandatory</code></td>
+    <td class="unsupported">No</td>
+    <td></td>
+  </tr>
+  <tr>
+    <td>Deprecate specifying multiple modifiers in the <code>map</code> clause without comma separator</td>
+    <td class="unsupported">No</td>
+    <td></td>
+  </tr>
+  <tr>
+    <td>Deprecate <code>destroy</code> clause on the <code>depobj</code> construct with no argument</td>
+    <td class="unsupported">No</td>
+    <td></td>
+  </tr>
 </tbody>
 </table>