diff mbox series

[wwwdocs,OpenACC] gcc-14/changes.html: OpenACC - mention support for first 2.7 features

Message ID f9847458-8898-4a5b-a461-659ea771fbec@codesourcery.com
State Unresolved
Headers
Series [wwwdocs,OpenACC] gcc-14/changes.html: OpenACC - mention support for first 2.7 features |

Checks

Context Check Description
snail/gcc-patch-check warning Git am fail log

Commit Message

Tobias Burnus Nov. 24, 2023, 4:22 p.m. UTC 
  Comments before I commit it?

Current version: https://gcc.gnu.org/gcc-14/changes.html#general

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

Comments

Gerald Pfeifer Nov. 26, 2023, 8:48 a.m. UTC | #1 
On Fri, 24 Nov 2023, Tobias Burnus wrote:
> Comments before I commit it?

+  <li id="openacc"><a href="https://gcc.gnu.org/wiki/OpenACC">OpenACC</a>
+    <li>OpenACC 2.7: The <code>self</code> clause was added to be used on
+      compute constructs and the <code>default</code> clause for data
+      constructs.<li>
+  </ul>
+  </li>

Where does that </ul> come from? I'm afraid this won't validate/render 
properly.

Gerald
Andreas Schwab Nov. 26, 2023, 9:18 a.m. UTC | #2 
On Nov 26 2023, Gerald Pfeifer wrote:

> On Fri, 24 Nov 2023, Tobias Burnus wrote:
>> Comments before I commit it?
>
> +  <li id="openacc"><a href="https://gcc.gnu.org/wiki/OpenACC">OpenACC</a>
> +    <li>OpenACC 2.7: The <code>self</code> clause was added to be used on
> +      compute constructs and the <code>default</code> clause for data
> +      constructs.<li>
                    </li>

> +  </ul>
> +  </li>
>
> Where does that </ul> come from? I'm afraid this won't validate/render 
> properly.

Neither that extra <li>, I think.
Tobias Burnus Nov. 27, 2023, 10:13 a.m. UTC | #3 
On 26.11.23 09:48, Gerald Pfeifer wrote:
> On Fri, 24 Nov 2023, Tobias Burnus wrote:
>> Comments before I commit it?
> +  <li id="openacc"><a href="https://gcc.gnu.org/wiki/OpenACC
> +    <li>OpenACC 2.7: The <code>self</code> clause was added to be used on
> +      compute constructs and the <code>default</code> clause for data
> +      constructs.<li>
> +  </ul>
> +  </li>
>
> Where does that </ul> come from? I'm afraid this won't validate/render
> properly.

That's the disadvantage of splitting patches from the same file ... The
'<ul>' starts / should start just before the quoted line, namely as
attached. (Updated patch attach - minor changes but syntactically relevant.)

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
diff mbox series

Patch

gcc-14/changes.html: OpenACC - mention support for first 2.7 features

diff --git a/htdocs/gcc-14/changes.html b/htdocs/gcc-14/changes.html
index 455798d1..c9d5f08a 100644
--- a/htdocs/gcc-14/changes.html
+++ b/htdocs/gcc-14/changes.html
@@ -81,4 +91,10 @@  a work-in-progress.</p>
   </ul>
   </li>
+  <li id="openacc"><a href="https://gcc.gnu.org/wiki/OpenACC">OpenACC</a>
+    <li>OpenACC 2.7: The <code>self</code> clause was added to be used on
+      compute constructs and the <code>default</code> clause for data
+      constructs.<li>
+  </ul>
+  </li>
   <li>For offload-device code generated via OpenMP and OpenACC, the math
       and the Fortran runtime libraries will now automatically be linked,