diff mbox series

[1/1] gcc-14: document P1689R5 scanning output support

Message ID 20231120162256.3620350-1-ben.boeckel@kitware.com
State New
Headers show
Series [1/1] gcc-14: document P1689R5 scanning output support | expand

Commit Message

Ben Boeckel Nov. 20, 2023, 4:22 p.m. UTC
---
 htdocs/gcc-14/changes.html | 11 +++++++++++
 1 file changed, 11 insertions(+)

Comments

Ben Boeckel Jan. 3, 2024, 3:34 p.m. UTC | #1
On Mon, Nov 20, 2023 at 11:22:56 -0500, Ben Boeckel wrote:
> ---
>  htdocs/gcc-14/changes.html | 11 +++++++++++
>  1 file changed, 11 insertions(+)
> 
> diff --git a/htdocs/gcc-14/changes.html b/htdocs/gcc-14/changes.html
> index 7278f753..b506eeb1 100644
> --- a/htdocs/gcc-14/changes.html
> +++ b/htdocs/gcc-14/changes.html
> @@ -112,6 +112,17 @@ a work-in-progress.</p>
>        </li>
>      </ul>
>    </li>
> +  <li>C++ module scanning for named modules is now available:
> +    <ul>
> +      <li><a href="https://wg21.link/P1689R5">P1689R5</a>, Format for
> +      describing dependencies of source files.
> +      </li>
> +      <li>The <code>-fdeps-format=</code>, <code>-fdeps-file=</code>, and
> +      <code>-fdeps=target=</code> flags may be used to generate P1689 output
> +      (the <code>p1689r5</code> format is the only available format today).
> +      </li>
> +    </ul>
> +  </li>
>  </ul>
>  
>  <h4 id="libstdcxx">Runtime Library (libstdc++)</h4>
> -- 
> 2.42.0

Ping? Is this the right place to submit this patch?

Thanks,

--Ben
Arsen Arsenović Jan. 6, 2024, 1:17 p.m. UTC | #2
Hi Ben,

Ben Boeckel <ben.boeckel@kitware.com> writes:

> Ping? Is this the right place to submit this patch?

Yes, this is the correct list, though it is usually recommended to use
--subject-prefix='PATCH wwwdocs' or such, to catch the right eyes.  See:
https://gcc.gnu.org/contribute.html#webchanges

I've added it to my subject, hopefully that works.

Have a lovely day!
--
Arsen Arsenović
Ben Boeckel April 27, 2024, 11:39 p.m. UTC | #3
On Sat, Jan 06, 2024 at 14:17:14 +0100, Arsen Arsenović wrote:
> Hi Ben,
> 
> Ben Boeckel <ben.boeckel@kitware.com> writes:
> 
> > Ping? Is this the right place to submit this patch?
> 
> Yes, this is the correct list, though it is usually recommended to use
> --subject-prefix='PATCH wwwdocs' or such, to catch the right eyes.  See:
> https://gcc.gnu.org/contribute.html#webchanges
> 
> I've added it to my subject, hopefully that works.

No bites yet… Anyone willing to review this patch so that it gets
mentioned on the website?

Thanks,

--Ben
Jonathan Wakely April 30, 2024, 9:24 a.m. UTC | #4
On 20/11/23 11:22 -0500, Ben Boeckel wrote:
>---
> htdocs/gcc-14/changes.html | 11 +++++++++++
> 1 file changed, 11 insertions(+)
>
>diff --git a/htdocs/gcc-14/changes.html b/htdocs/gcc-14/changes.html
>index 7278f753..b506eeb1 100644
>--- a/htdocs/gcc-14/changes.html
>+++ b/htdocs/gcc-14/changes.html
>@@ -112,6 +112,17 @@ a work-in-progress.</p>
>       </li>
>     </ul>
>   </li>
>+  <li>C++ module scanning for named modules is now available:
>+    <ul>
>+      <li><a href="https://wg21.link/P1689R5">P1689R5</a>, Format for
>+      describing dependencies of source files.
>+      </li>
>+      <li>The <code>-fdeps-format=</code>, <code>-fdeps-file=</code>, and
>+      <code>-fdeps=target=</code> flags may be used to generate P1689 output

This should be -fdeps-target= not -fdeps=target=.


>+      (the <code>p1689r5</code> format is the only available format today).

I wish the option was more descriptive than "p1689r5", which nobody is
going to remember (but I assume we don't actually need to specify it
explicitly since it's the only supported format).

>+      </li>
>+    </ul>

Do we need a list for this item? It seems a bit weird that the first
list item is just the paper  How about just a single paragraph?

C++ module scanning for named modules is now available, based on the
format described in <a href="https://wg21.link/P1689R5">P1689R5</a>,
Format for describing dependencies of source files. The
<code>-fdeps-format=</code>, <code>-fdeps-file=</code>, and
<code>-fdeps-target=</code> flags may be used to generate dependency
information. In GCC 14 <code>p1689r5</code> is the only valid argument
for <code>-fdeps-format=</code>.


>+  </li>
> </ul>
>
> <h4 id="libstdcxx">Runtime Library (libstdc++)</h4>
>-- 
>2.42.0
>
Ben Boeckel May 2, 2024, 6:03 p.m. UTC | #5
On Tue, Apr 30, 2024 at 10:24:44 +0100, Jonathan Wakely wrote:
> On 20/11/23 11:22 -0500, Ben Boeckel wrote:
> >---
> > htdocs/gcc-14/changes.html | 11 +++++++++++
> > 1 file changed, 11 insertions(+)
> >
> >diff --git a/htdocs/gcc-14/changes.html b/htdocs/gcc-14/changes.html
> >index 7278f753..b506eeb1 100644
> >--- a/htdocs/gcc-14/changes.html
> >+++ b/htdocs/gcc-14/changes.html
> >@@ -112,6 +112,17 @@ a work-in-progress.</p>
> >       </li>
> >     </ul>
> >   </li>
> >+  <li>C++ module scanning for named modules is now available:
> >+    <ul>
> >+      <li><a href="https://wg21.link/P1689R5">P1689R5</a>, Format for
> >+      describing dependencies of source files.
> >+      </li>
> >+      <li>The <code>-fdeps-format=</code>, <code>-fdeps-file=</code>, and
> >+      <code>-fdeps=target=</code> flags may be used to generate P1689 output
> 
> This should be -fdeps-target= not -fdeps=target=.

Whoops, yep.

> >+      (the <code>p1689r5</code> format is the only available format today).
> 
> I wish the option was more descriptive than "p1689r5", which nobody is
> going to remember (but I assume we don't actually need to specify it
> explicitly since it's the only supported format).

All users of the flag should be having it in the build system itself;
hand-coded makefiles can use it, but will need considerable `jq`
gymnastics to turn the output into properly understood make syntax
snippets for their build.

> >+      </li>
> >+    </ul>
> 
> Do we need a list for this item? It seems a bit weird that the first
> list item is just the paper  How about just a single paragraph?
> 
> C++ module scanning for named modules is now available, based on the
> format described in <a href="https://wg21.link/P1689R5">P1689R5</a>,
> Format for describing dependencies of source files. The
> <code>-fdeps-format=</code>, <code>-fdeps-file=</code>, and
> <code>-fdeps-target=</code> flags may be used to generate dependency
> information. In GCC 14 <code>p1689r5</code> is the only valid argument
> for <code>-fdeps-format=</code>.

Sounds good. New patch attached.

--Ben
From d973efa9689db7d46211721e7c00feea7e6445a6 Mon Sep 17 00:00:00 2001
From: Ben Boeckel <ben.boeckel@kitware.com>
Date: Thu, 2 May 2024 14:00:01 -0400
Subject: [PATCH 1/1] gcc-14: document P1689R5 scanning output support

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

diff --git a/htdocs/gcc-14/changes.html b/htdocs/gcc-14/changes.html
index 8dfbf7dc..8998e6c0 100644
--- a/htdocs/gcc-14/changes.html
+++ b/htdocs/gcc-14/changes.html
@@ -513,6 +513,12 @@ a work-in-progress.</p>
   <li>GCC supports a new pragma <code>&#x23;pragma GCC novector</code> to
       indicate to the vectorizer not to vectorize the loop annotated with the
       pragma.</li>
+  <li>C++ module scanning for named modules is now available, based on the
+    format described in<a href="https://wg21.link/P1689R5">P1689R5</a>, Format for
+    describing dependencies of source files. The <code>-fdeps-format=</code>,
+    <code>-fdeps-file=</code>, and <code>-fdeps-target=</code> flags may be
+    used to generate dependency information. In GCC 14 <code>p1689r5</code> is
+    the only valid argument for <code>-fdeps-format=</code>.</li>
 </ul>
 
 <h4 id="libstdcxx">Runtime Library (libstdc++)</h4>
Jonathan Wakely May 7, 2024, 10:05 a.m. UTC | #6
On Thu, 2 May 2024 at 19:03, Ben Boeckel <ben.boeckel@kitware.com> wrote:
>
> On Tue, Apr 30, 2024 at 10:24:44 +0100, Jonathan Wakely wrote:
> > On 20/11/23 11:22 -0500, Ben Boeckel wrote:
> > >---
> > > htdocs/gcc-14/changes.html | 11 +++++++++++
> > > 1 file changed, 11 insertions(+)
> > >
> > >diff --git a/htdocs/gcc-14/changes.html b/htdocs/gcc-14/changes.html
> > >index 7278f753..b506eeb1 100644
> > >--- a/htdocs/gcc-14/changes.html
> > >+++ b/htdocs/gcc-14/changes.html
> > >@@ -112,6 +112,17 @@ a work-in-progress.</p>
> > >       </li>
> > >     </ul>
> > >   </li>
> > >+  <li>C++ module scanning for named modules is now available:
> > >+    <ul>
> > >+      <li><a href="https://wg21.link/P1689R5">P1689R5</a>, Format for
> > >+      describing dependencies of source files.
> > >+      </li>
> > >+      <li>The <code>-fdeps-format=</code>, <code>-fdeps-file=</code>, and
> > >+      <code>-fdeps=target=</code> flags may be used to generate P1689 output
> >
> > This should be -fdeps-target= not -fdeps=target=.
>
> Whoops, yep.
>
> > >+      (the <code>p1689r5</code> format is the only available format today).
> >
> > I wish the option was more descriptive than "p1689r5", which nobody is
> > going to remember (but I assume we don't actually need to specify it
> > explicitly since it's the only supported format).
>
> All users of the flag should be having it in the build system itself;
> hand-coded makefiles can use it, but will need considerable `jq`
> gymnastics to turn the output into properly understood make syntax
> snippets for their build.
>
> > >+      </li>
> > >+    </ul>
> >
> > Do we need a list for this item? It seems a bit weird that the first
> > list item is just the paper  How about just a single paragraph?
> >
> > C++ module scanning for named modules is now available, based on the
> > format described in <a href="https://wg21.link/P1689R5">P1689R5</a>,
> > Format for describing dependencies of source files. The
> > <code>-fdeps-format=</code>, <code>-fdeps-file=</code>, and
> > <code>-fdeps-target=</code> flags may be used to generate dependency
> > information. In GCC 14 <code>p1689r5</code> is the only valid argument
> > for <code>-fdeps-format=</code>.
>
> Sounds good. New patch attached.

Thanks, Ben. I reformatted the whitespace so all the lines were under
80 columns and pushed it to wwwdocs.
diff mbox series

Patch

diff --git a/htdocs/gcc-14/changes.html b/htdocs/gcc-14/changes.html
index 7278f753..b506eeb1 100644
--- a/htdocs/gcc-14/changes.html
+++ b/htdocs/gcc-14/changes.html
@@ -112,6 +112,17 @@  a work-in-progress.</p>
       </li>
     </ul>
   </li>
+  <li>C++ module scanning for named modules is now available:
+    <ul>
+      <li><a href="https://wg21.link/P1689R5">P1689R5</a>, Format for
+      describing dependencies of source files.
+      </li>
+      <li>The <code>-fdeps-format=</code>, <code>-fdeps-file=</code>, and
+      <code>-fdeps=target=</code> flags may be used to generate P1689 output
+      (the <code>p1689r5</code> format is the only available format today).
+      </li>
+    </ul>
+  </li>
 </ul>
 
 <h4 id="libstdcxx">Runtime Library (libstdc++)</h4>