diff mbox series

[5/7] libstdc++: printf format string fix in testsuite

Message ID	3557908.R56niFO833@minbar
State	Accepted
Headers	Received-SPF: pass (google.com: domain of gcc-patches-bounces+ouuuleilei=gmail.com@gcc.gnu.org designates 8.43.85.97 as permitted sender) client-ip=8.43.85.97; DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org C10AF385B539 To: <gcc-patches@gcc.gnu.org>, <libstdc++@gcc.gnu.org> Subject: [PATCH 5/7] libstdc++: printf format string fix in testsuite Date: Wed, 15 Feb 2023 21:50:03 +0100 Message-ID: <3557908.R56niFO833@minbar> Organization: GSI Helmholtz Centre for Heavy Ion Research In-Reply-To: <3238840.44csPzL39Z@minbar> References: <3238840.44csPzL39Z@minbar> MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="nextPart3681468.MHq7AAxBmi" Content-Transfer-Encoding: 7Bit Precedence: list From: Matthias Kretz via Gcc-patches <gcc-patches@gcc.gnu.org> Reply-To: Matthias Kretz <m.kretz@gsi.de> Errors-To: gcc-patches-bounces+ouuuleilei=gmail.com@gcc.gnu.org Sender: "Gcc-patches" <gcc-patches-bounces+ouuuleilei=gmail.com@gcc.gnu.org> X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?=
Series	Work on PR108030 and several simd bugfixes and testsuite improvements \| [0/7] Work on PR108030 and several simd bugfixes and testsuite improvements [1/7] libstdc++: Ensure __builtin_constant_p isn't lost on the way [2/7] libstdc++: Annotate most lambdas with always_inline [3/7] libstdc++: Document timeout and timeout-factor of simd tests [4/7] libstdc++: Use a PCH to speed up check-simd [5/7] libstdc++: printf format string fix in testsuite [6/7] libstdc++: Fix incorrect __builtin_is_constant_evaluated calls [7/7] libstdc++: Fix incorrect function call in -ffast-math optimization

Checks

Context	Check	Description
snail/gcc-patch-check	success	Github commit url

Commit Message

Matthias Kretz Feb. 15, 2023, 8:50 p.m. UTC

  Signed-off-by: Matthias Kretz <m.kretz@gsi.de>

libstdc++-v3/ChangeLog:

	* testsuite/experimental/simd/tests/bits/verify.h
	(verify::verify): Use %zx for size_t in format string.
---
 libstdc++-v3/testsuite/experimental/simd/tests/bits/verify.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


--
──────────────────────────────────────────────────────────────────────────
 Dr. Matthias Kretz                           https://mattkretz.github.io
 GSI Helmholtz Centre for Heavy Ion Research               https://gsi.de
 stdₓ::simd
──────────────────────────────────────────────────────────────────────────

Comments

Jonathan Wakely Feb. 15, 2023, 11:16 p.m. UTC | #1

On Wed, 15 Feb 2023 at 20:52, Matthias Kretz via Libstdc++
<libstdc++@gcc.gnu.org> wrote:
>
>
>
> Signed-off-by: Matthias Kretz <m.kretz@gsi.de>
>
> libstdc++-v3/ChangeLog:
>
>         * testsuite/experimental/simd/tests/bits/verify.h
>         (verify::verify): Use %zx for size_t in format string.
> ---
>  libstdc++-v3/testsuite/experimental/simd/tests/bits/verify.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

I think this one would be OK to commit without approval, as "obvious".

OK for all relevant branches anyway.

diff mbox series

Patch

diff --git a/libstdc++-v3/testsuite/experimental/simd/tests/bits/verify.h b/libstdc++-v3/testsuite/experimental/simd/tests/bits/verify.h
index 2ab3ad3fa8c..01ad50bd01a 100644
--- a/libstdc++-v3/testsuite/experimental/simd/tests/bits/verify.h
+++ b/libstdc++-v3/testsuite/experimental/simd/tests/bits/verify.h
@@ -137,7 +137,7 @@  public:
     {
       if (m_failed)
 	[&] {
-	  __builtin_fprintf(stderr, "%s:%d: (%s):\nInstruction Pointer: %x\n"
+	  __builtin_fprintf(stderr, "%s:%d: (%s):\nInstruction Pointer: %zx\n"
 				    "Assertion '%s' failed.\n",
 			    file, line, func, m_ip, cond);
 	  (print(extra_info, int()), ...);