diff mbox series

gcc-wwwdocs: Mention check-c++-all target for C++ front end patch testing

Message ID 010201924c6d808d-df5ec459-1447-4aea-8862-a59371704b8b-000000@eu-west-1.amazonses.com
State New
Headers show
Series gcc-wwwdocs: Mention check-c++-all target for C++ front end patch testing | expand

Commit Message

Simon Martin Oct. 2, 2024, 8:51 a.m. UTC
This is a follow-up to the discussion about testing changes to the C++
front end in
https://gcc.gnu.org/pipermail/gcc-patches/2024-October/664258.html

It also clarifies that the make invocation examples should be made from
the *build* tree.

Validated fine via https://validator.w3.org.
---
 htdocs/contribute.html | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

Comments

Jason Merrill Oct. 4, 2024, 3:06 a.m. UTC | #1
On 10/2/24 4:51 AM, Simon Martin wrote:
> This is a follow-up to the discussion about testing changes to the C++
> front end in
> https://gcc.gnu.org/pipermail/gcc-patches/2024-October/664258.html
> 
> It also clarifies that the make invocation examples should be made from
> the *build* tree.
> 
> Validated fine via https://validator.w3.org.

OK.

> ---
>   htdocs/contribute.html | 10 +++++++++-
>   1 file changed, 9 insertions(+), 1 deletion(-)
> 
> diff --git a/htdocs/contribute.html b/htdocs/contribute.html
> index 53c27c6e..3ab65323 100644
> --- a/htdocs/contribute.html
> +++ b/htdocs/contribute.html
> @@ -111,9 +111,17 @@ For a normal native configuration, running</p>
>   make bootstrap
>   make -k check
>   </pre></blockquote>
> -<p>from the top level of the GCC tree (<strong>not</strong> the
> +<p>from the top level of the GCC build tree (<strong>not</strong> the
>   <code>gcc</code> subdirectory) will accomplish this.</p>
>   
> +<p>If your change is to the C++ front end, you need to run the C++ testsuite
> +in all standard conformance levels. For a normal native configuration,
> +running</p>
> +<blockquote><pre>
> +make -C gcc -k check-c++-all
> +</pre></blockquote>
> +<p>from the top level of the GCC build tree will accomplish this.</p>
> +
>   <p>If your change is to a front end other than the C or C++ front end,
>   or a runtime library other than <code>libgcc</code>, you need to verify
>   only that the runtime library for that language still builds and the
Sam James Oct. 4, 2024, 3:10 a.m. UTC | #2
Simon Martin <simon@nasilyan.com> writes:

> This is a follow-up to the discussion about testing changes to the C++
> front end in
> https://gcc.gnu.org/pipermail/gcc-patches/2024-October/664258.html
>
> It also clarifies that the make invocation examples should be made from
> the *build* tree.
>
> Validated fine via https://validator.w3.org.

I've added this to the wiki at https://gcc.gnu.org/wiki/Testing_GCC too, thanks.

> ---
>  htdocs/contribute.html | 10 +++++++++-
>  1 file changed, 9 insertions(+), 1 deletion(-)
>
> [...]

thanks,
sam
diff mbox series

Patch

diff --git a/htdocs/contribute.html b/htdocs/contribute.html
index 53c27c6e..3ab65323 100644
--- a/htdocs/contribute.html
+++ b/htdocs/contribute.html
@@ -111,9 +111,17 @@  For a normal native configuration, running</p>
 make bootstrap
 make -k check
 </pre></blockquote>
-<p>from the top level of the GCC tree (<strong>not</strong> the
+<p>from the top level of the GCC build tree (<strong>not</strong> the
 <code>gcc</code> subdirectory) will accomplish this.</p>
 
+<p>If your change is to the C++ front end, you need to run the C++ testsuite
+in all standard conformance levels. For a normal native configuration,
+running</p>
+<blockquote><pre>
+make -C gcc -k check-c++-all
+</pre></blockquote>
+<p>from the top level of the GCC build tree will accomplish this.</p>
+
 <p>If your change is to a front end other than the C or C++ front end,
 or a runtime library other than <code>libgcc</code>, you need to verify
 only that the runtime library for that language still builds and the