On Sat, May 8, 2021 at 7:18 AM H.J. Lu <hjl.to...@gmail.com> wrote:
>
> PR tree-optimization/42587
> * gcc.target/i386/pr42587.c: New test.
> ---
> gcc/testsuite/gcc.target/i386/pr42587.c | 35 +++++++++++++++++++++++++
> 1 file changed, 35 insertions(+)
> create mode 100644 gcc/testsuite/gcc.target/i386/pr42587.c
>
> diff --git a/gcc/testsuite/gcc.target/i386/pr42587.c
> b/gcc/testsuite/gcc.target/i386/pr42587.c
> new file mode 100644
> index 00000000000..d4b5143eb6d
> --- /dev/null
> +++ b/gcc/testsuite/gcc.target/i386/pr42587.c
> @@ -0,0 +1,35 @@
> +/* { dg-do compile } */
> +/* { dg-options "-O2 -fdump-tree-store-merging" } */
> +
> +typedef unsigned char u8;
> +typedef unsigned int u32;
> +union __anonunion_out_195
> +{
> + u32 value;
> + u8 bytes[4];
> +};
> +union __anonunion_in_196
> +{
> + u32 value;
> + u8 bytes[4];
> +};
> +extern void acpi_ut_track_stack_ptr (void);
> +u32 acpi_ut_dword_byte_swap (u32 value);
> +u32
> +acpi_ut_dword_byte_swap (u32 value)
> +{
> + union __anonunion_out_195 out;
> + union __anonunion_in_196 in;
> +
> + {
> + acpi_ut_track_stack_ptr ();
> + in.value = value;
> + out.bytes[0] = in.bytes[3];
> + out.bytes[1] = in.bytes[2];
> + out.bytes[2] = in.bytes[1];
> + out.bytes[3] = in.bytes[0];
> + return (out.value);
> + }
> +}
> +
> +/* { dg-final { scan-tree-dump "32 bit bswap implementation found at"
> "store-merging" } } */
> --
> 2.31.1
>
The v2 patch to add gcc.dg/optimize-bswapsi-6.c. Which one is
preferred?
--
H.J.
From 33804e2bf14605a4676de3b3784ada406fdbaa21 Mon Sep 17 00:00:00 2001
From: "H.J. Lu" <hjl.tools@gmail.com>
Date: Sat, 8 May 2021 07:23:25 -0700
Subject: [PATCH v2] Add a test for PR tree-optimization/42587
PR tree-optimization/42587
* gcc.dg/optimize-bswapsi-6.c: New test.
---
gcc/testsuite/gcc.dg/optimize-bswapsi-6.c | 38 +++++++++++++++++++++++
1 file changed, 38 insertions(+)
create mode 100644 gcc/testsuite/gcc.dg/optimize-bswapsi-6.c
diff --git a/gcc/testsuite/gcc.dg/optimize-bswapsi-6.c b/gcc/testsuite/gcc.dg/optimize-bswapsi-6.c
new file mode 100644
index 00000000000..3c089b36fd8
--- /dev/null
+++ b/gcc/testsuite/gcc.dg/optimize-bswapsi-6.c
@@ -0,0 +1,38 @@
+/* PR tree-optimization/42587 */
+/* { dg-do compile } */
+/* { dg-require-effective-target bswap } */
+/* { dg-options "-O2 -fdump-tree-store-merging" } */
+/* { dg-additional-options "-march=z900" { target s390-*-* } } */
+
+typedef unsigned char u8;
+typedef unsigned int u32;
+union __anonunion_out_195
+{
+ u32 value;
+ u8 bytes[4];
+};
+union __anonunion_in_196
+{
+ u32 value;
+ u8 bytes[4];
+};
+extern void acpi_ut_track_stack_ptr (void);
+u32 acpi_ut_dword_byte_swap (u32 value);
+u32
+acpi_ut_dword_byte_swap (u32 value)
+{
+ union __anonunion_out_195 out;
+ union __anonunion_in_196 in;
+
+ {
+ acpi_ut_track_stack_ptr ();
+ in.value = value;
+ out.bytes[0] = in.bytes[3];
+ out.bytes[1] = in.bytes[2];
+ out.bytes[2] = in.bytes[1];
+ out.bytes[3] = in.bytes[0];
+ return (out.value);
+ }
+}
+
+/* { dg-final { scan-tree-dump "32 bit bswap implementation found at" "store-merging" } } */
--
2.31.1
