public inbox for gcc-cvs@sourceware.org
help / color / mirror / Atom feed
* [gcc(refs/users/meissner/heads/dmf010)] Revert patches
@ 2023-03-09 16:52 Michael Meissner
0 siblings, 0 replies; 3+ messages in thread
From: Michael Meissner @ 2023-03-09 16:52 UTC (permalink / raw)
To: gcc-cvs
https://gcc.gnu.org/g:b3a3dee4fac59c8231d558ea91c2ec146d10a979
commit b3a3dee4fac59c8231d558ea91c2ec146d10a979
Author: Michael Meissner <meissner@linux.ibm.com>
Date: Thu Mar 9 11:52:31 2023 -0500
Revert patches
Diff:
---
gcc/config/rs6000/rs6000.cc | 111 ++++++++++++----------------
gcc/testsuite/gcc.target/powerpc/divic3-1.c | 21 ------
gcc/testsuite/gcc.target/powerpc/divic3-2.c | 25 -------
gcc/testsuite/gcc.target/powerpc/mulic3-1.c | 21 ------
gcc/testsuite/gcc.target/powerpc/mulic3-2.c | 25 -------
5 files changed, 47 insertions(+), 156 deletions(-)
diff --git a/gcc/config/rs6000/rs6000.cc b/gcc/config/rs6000/rs6000.cc
index b1b35b4bb77..8e0b0d022db 100644
--- a/gcc/config/rs6000/rs6000.cc
+++ b/gcc/config/rs6000/rs6000.cc
@@ -11154,6 +11154,26 @@ init_float128_ibm (machine_mode mode)
}
}
+/* Create a decl for either complex long double multiply or complex long double
+ divide when long double is IEEE 128-bit floating point. We can't use
+ __multc3 and __divtc3 because the original long double using IBM extended
+ double used those names. The complex multiply/divide functions are encoded
+ as builtin functions with a complex result and 4 scalar inputs. */
+
+static void
+create_complex_muldiv (const char *name, built_in_function fncode, tree fntype)
+{
+ tree fndecl = add_builtin_function (name, fntype, fncode, BUILT_IN_NORMAL,
+ name, NULL_TREE);
+
+ set_builtin_decl (fncode, fndecl, true);
+
+ if (TARGET_DEBUG_BUILTIN)
+ fprintf (stderr, "create complex %s, fncode: %d\n", name, (int) fncode);
+
+ return;
+}
+
/* Set up IEEE 128-bit floating point routines. Use different names if the
arguments can be passed in a vector register. The historical PowerPC
implementation of IEEE 128-bit floating point used _q_<op> for the names, so
@@ -11165,6 +11185,32 @@ init_float128_ieee (machine_mode mode)
{
if (FLOAT128_VECTOR_P (mode))
{
+ static bool complex_muldiv_init_p = false;
+
+ /* Set up to call __mulkc3 and __divkc3 under -mabi=ieeelongdouble. If
+ we have clone or target attributes, this will be called a second
+ time. We want to create the built-in function only once. */
+ if (mode == TFmode && TARGET_IEEEQUAD && !complex_muldiv_init_p)
+ {
+ complex_muldiv_init_p = true;
+ built_in_function fncode_mul =
+ (built_in_function) (BUILT_IN_COMPLEX_MUL_MIN + TCmode
+ - MIN_MODE_COMPLEX_FLOAT);
+ built_in_function fncode_div =
+ (built_in_function) (BUILT_IN_COMPLEX_DIV_MIN + TCmode
+ - MIN_MODE_COMPLEX_FLOAT);
+
+ tree fntype = build_function_type_list (complex_long_double_type_node,
+ long_double_type_node,
+ long_double_type_node,
+ long_double_type_node,
+ long_double_type_node,
+ NULL_TREE);
+
+ create_complex_muldiv ("__mulkc3", fncode_mul, fntype);
+ create_complex_muldiv ("__divkc3", fncode_div, fntype);
+ }
+
set_optab_libfunc (add_optab, mode, "__addkf3");
set_optab_libfunc (sub_optab, mode, "__subkf3");
set_optab_libfunc (neg_optab, mode, "__negkf2");
@@ -28182,27 +28228,6 @@ rs6000_starting_frame_offset (void)
return RS6000_STARTING_FRAME_OFFSET;
}
\f
-/* Internal function to return the built-in function id for the complex
- multiply operation for a given mode. */
-
-static inline built_in_function
-complex_multiply_builtin_code (machine_mode mode)
-{
- gcc_assert (IN_RANGE (mode, MIN_MODE_COMPLEX_FLOAT, MAX_MODE_COMPLEX_FLOAT));
- return (built_in_function) (BUILT_IN_COMPLEX_MUL_MIN + mode
- - MIN_MODE_COMPLEX_FLOAT);
-}
-
-/* Internal function to return the built-in function id for the complex divide
- operation for a given mode. */
-
-static inline built_in_function
-complex_divide_builtin_code (machine_mode mode)
-{
- gcc_assert (IN_RANGE (mode, MIN_MODE_COMPLEX_FLOAT, MAX_MODE_COMPLEX_FLOAT));
- return (built_in_function) (BUILT_IN_COMPLEX_DIV_MIN + mode
- - MIN_MODE_COMPLEX_FLOAT);
-}
/* On 64-bit Linux and Freebsd systems, possibly switch the long double library
function names from <foo>l to <foo>f128 if the default long double type is
@@ -28221,53 +28246,11 @@ complex_divide_builtin_code (machine_mode mode)
only do this transformation if the __float128 type is enabled. This
prevents us from doing the transformation on older 32-bit ports that might
have enabled using IEEE 128-bit floating point as the default long double
- type.
-
- We also use the TARGET_MANGLE_DECL_ASSEMBLER_NAME hook to change the
- function names used for complex multiply and divide to the appropriate
- names. */
+ type. */
static tree
rs6000_mangle_decl_assembler_name (tree decl, tree id)
{
- /* Handle complex multiply/divide. For IEEE 128-bit, use __mulkc3 or
- __divkc3 and for IBM 128-bit use __multc3 and __divtc3. */
- if (TARGET_FLOAT128_TYPE
- && TREE_CODE (decl) == FUNCTION_DECL
- && DECL_IS_UNDECLARED_BUILTIN (decl)
- && DECL_BUILT_IN_CLASS (decl) == BUILT_IN_NORMAL)
- {
- built_in_function id = DECL_FUNCTION_CODE (decl);
- const char *newname = NULL;
-
- if (id == complex_multiply_builtin_code (KCmode))
- newname = "__mulkc3";
-
- else if (id == complex_multiply_builtin_code (ICmode))
- newname = "__multc3";
-
- else if (id == complex_multiply_builtin_code (TCmode))
- newname = (TARGET_IEEEQUAD) ? "__mulkc3" : "__multc3";
-
- else if (id == complex_divide_builtin_code (KCmode))
- newname = "__divkc3";
-
- else if (id == complex_divide_builtin_code (ICmode))
- newname = "__divtc3";
-
- else if (id == complex_divide_builtin_code (TCmode))
- newname = (TARGET_IEEEQUAD) ? "__divkc3" : "__divtc3";
-
- if (newname)
- {
- if (TARGET_DEBUG_BUILTIN)
- fprintf (stderr, "Map complex mul/div => %s\n", newname);
-
- return get_identifier (newname);
- }
- }
-
- /* Map long double built-in functions if long double is IEEE 128-bit. */
if (TARGET_FLOAT128_TYPE && TARGET_IEEEQUAD && TARGET_LONG_DOUBLE_128
&& TREE_CODE (decl) == FUNCTION_DECL
&& DECL_IS_UNDECLARED_BUILTIN (decl)
diff --git a/gcc/testsuite/gcc.target/powerpc/divic3-1.c b/gcc/testsuite/gcc.target/powerpc/divic3-1.c
deleted file mode 100644
index 8bf188617aa..00000000000
--- a/gcc/testsuite/gcc.target/powerpc/divic3-1.c
+++ /dev/null
@@ -1,21 +0,0 @@
-/* { dg-do compile } */
-/* { dg-require-effective-target ppc_float128_sw } */
-/* { dg-options "-O2 -mabi=ieeelongdouble" } */
-
-/* When GCC is configured with an older library that does not support IEEE
- 128-bit, it issues a warning if you change the long double type. We use
- -Wno-psabi to silence this warning. Since this is a code generation test,
- it does not matter if the library has full IEEE 128-bit support. */
-
-/* Check that complex divide generates the right call for __ibm128 when long
- double is IEEE 128-bit floating point. */
-
-typedef _Complex long double c_ibm128_t __attribute__((mode(__IC__)));
-
-void
-divide (c_ibm128_t *p, c_ibm128_t *q, c_ibm128_t *r)
-{
- *p = *q / *r;
-}
-
-/* { dg-final { scan-assembler "bl __divtc3" } } */
diff --git a/gcc/testsuite/gcc.target/powerpc/divic3-2.c b/gcc/testsuite/gcc.target/powerpc/divic3-2.c
deleted file mode 100644
index f0efb76ec7d..00000000000
--- a/gcc/testsuite/gcc.target/powerpc/divic3-2.c
+++ /dev/null
@@ -1,25 +0,0 @@
-/* { dg-do compile } */
-/* { dg-require-effective-target ppc_float128_sw } */
-/* { dg-require-effective-target longdouble128 } */
-/* { dg-options "-O2 -mabi=ibmlongdouble" } */
-
-/* When GCC is configured with an older library that does not support IEEE
- 128-bit, it issues a warning if you change the long double type. We use
- -Wno-psabi to silence this warning. Since this is a code generation test,
- it does not matter if the library has full IEEE 128-bit support.
-
- We also need to require that the default long double is 128-bits, otherwise
- the TC/TF modes might not be available. */
-
-/* Check that complex divide generates the right call for __ibm128 when long
- double is IBM 128-bit floating point. */
-
-typedef _Complex long double c_ibm128_t __attribute__((mode(__TC__)));
-
-void
-divide (c_ibm128_t *p, c_ibm128_t *q, c_ibm128_t *r)
-{
- *p = *q / *r;
-}
-
-/* { dg-final { scan-assembler "bl __divtc3" } } */
diff --git a/gcc/testsuite/gcc.target/powerpc/mulic3-1.c b/gcc/testsuite/gcc.target/powerpc/mulic3-1.c
deleted file mode 100644
index ac23c3e3437..00000000000
--- a/gcc/testsuite/gcc.target/powerpc/mulic3-1.c
+++ /dev/null
@@ -1,21 +0,0 @@
-/* { dg-do compile } */
-/* { dg-require-effective-target ppc_float128_sw } */
-/* { dg-options "-O2 -mabi=ieeelongdouble" } */
-
-/* When GCC is configured with an older library that does not support IEEE
- 128-bit, it issues a warning if you change the long double type. We use
- -Wno-psabi to silence this warning. Since this is a code generation test,
- it does not matter if the library has full IEEE 128-bit support. */
-
-/* Check that complex multiply generates the right call for __ibm128 when long
- double is IEEE 128-bit floating point. */
-
-typedef _Complex long double c_ibm128_t __attribute__((mode(__IC__)));
-
-void
-multiply (c_ibm128_t *p, c_ibm128_t *q, c_ibm128_t *r)
-{
- *p = *q * *r;
-}
-
-/* { dg-final { scan-assembler "__multc3" } } */
diff --git a/gcc/testsuite/gcc.target/powerpc/mulic3-2.c b/gcc/testsuite/gcc.target/powerpc/mulic3-2.c
deleted file mode 100644
index 75355160596..00000000000
--- a/gcc/testsuite/gcc.target/powerpc/mulic3-2.c
+++ /dev/null
@@ -1,25 +0,0 @@
-/* { dg-do compile } */
-/* { dg-require-effective-target ppc_float128_sw } */
-/* { dg-require-effective-target longdouble128 } */
-/* { dg-options "-O2 -mabi=ibmlongdouble" } */
-
-/* When GCC is configured with an older library that does not support IEEE
- 128-bit, it issues a warning if you change the long double type. We use
- -Wno-psabi to silence this warning. Since this is a code generation test,
- it does not matter if the library has full IEEE 128-bit support.
-
- We also need to require that the default long double is 128-bits, otherwise
- the TC/TF modes might not be available. */
-
-/* Check that complex multiply generates the right call for __ibm128 when long
- double is IBM 128-bit floating point. */
-
-typedef _Complex long double c_ibm128_t __attribute__((mode(__TC__)));
-
-void
-multiply (c_ibm128_t *p, c_ibm128_t *q, c_ibm128_t *r)
-{
- *p = *q * *r;
-}
-
-/* { dg-final { scan-assembler "__multc3" } } */
^ permalink raw reply [flat|nested] 3+ messages in thread
* [gcc(refs/users/meissner/heads/dmf010)] Revert patches
@ 2023-03-09 23:26 Michael Meissner
0 siblings, 0 replies; 3+ messages in thread
From: Michael Meissner @ 2023-03-09 23:26 UTC (permalink / raw)
To: gcc-cvs
https://gcc.gnu.org/g:e0b2de4c4093151abde53aad98c870bc6777ef41
commit e0b2de4c4093151abde53aad98c870bc6777ef41
Author: Michael Meissner <meissner@linux.ibm.com>
Date: Thu Mar 9 18:26:26 2023 -0500
Revert patches
Diff:
---
gcc/config/rs6000/rs6000.cc | 111 ++++++++++++----------------
gcc/testsuite/gcc.target/powerpc/divic3-1.c | 21 ------
gcc/testsuite/gcc.target/powerpc/divic3-2.c | 25 -------
gcc/testsuite/gcc.target/powerpc/mulic3-1.c | 21 ------
gcc/testsuite/gcc.target/powerpc/mulic3-2.c | 25 -------
5 files changed, 47 insertions(+), 156 deletions(-)
diff --git a/gcc/config/rs6000/rs6000.cc b/gcc/config/rs6000/rs6000.cc
index fa5f93a874f..8e0b0d022db 100644
--- a/gcc/config/rs6000/rs6000.cc
+++ b/gcc/config/rs6000/rs6000.cc
@@ -11154,6 +11154,26 @@ init_float128_ibm (machine_mode mode)
}
}
+/* Create a decl for either complex long double multiply or complex long double
+ divide when long double is IEEE 128-bit floating point. We can't use
+ __multc3 and __divtc3 because the original long double using IBM extended
+ double used those names. The complex multiply/divide functions are encoded
+ as builtin functions with a complex result and 4 scalar inputs. */
+
+static void
+create_complex_muldiv (const char *name, built_in_function fncode, tree fntype)
+{
+ tree fndecl = add_builtin_function (name, fntype, fncode, BUILT_IN_NORMAL,
+ name, NULL_TREE);
+
+ set_builtin_decl (fncode, fndecl, true);
+
+ if (TARGET_DEBUG_BUILTIN)
+ fprintf (stderr, "create complex %s, fncode: %d\n", name, (int) fncode);
+
+ return;
+}
+
/* Set up IEEE 128-bit floating point routines. Use different names if the
arguments can be passed in a vector register. The historical PowerPC
implementation of IEEE 128-bit floating point used _q_<op> for the names, so
@@ -11165,6 +11185,32 @@ init_float128_ieee (machine_mode mode)
{
if (FLOAT128_VECTOR_P (mode))
{
+ static bool complex_muldiv_init_p = false;
+
+ /* Set up to call __mulkc3 and __divkc3 under -mabi=ieeelongdouble. If
+ we have clone or target attributes, this will be called a second
+ time. We want to create the built-in function only once. */
+ if (mode == TFmode && TARGET_IEEEQUAD && !complex_muldiv_init_p)
+ {
+ complex_muldiv_init_p = true;
+ built_in_function fncode_mul =
+ (built_in_function) (BUILT_IN_COMPLEX_MUL_MIN + TCmode
+ - MIN_MODE_COMPLEX_FLOAT);
+ built_in_function fncode_div =
+ (built_in_function) (BUILT_IN_COMPLEX_DIV_MIN + TCmode
+ - MIN_MODE_COMPLEX_FLOAT);
+
+ tree fntype = build_function_type_list (complex_long_double_type_node,
+ long_double_type_node,
+ long_double_type_node,
+ long_double_type_node,
+ long_double_type_node,
+ NULL_TREE);
+
+ create_complex_muldiv ("__mulkc3", fncode_mul, fntype);
+ create_complex_muldiv ("__divkc3", fncode_div, fntype);
+ }
+
set_optab_libfunc (add_optab, mode, "__addkf3");
set_optab_libfunc (sub_optab, mode, "__subkf3");
set_optab_libfunc (neg_optab, mode, "__negkf2");
@@ -28182,27 +28228,6 @@ rs6000_starting_frame_offset (void)
return RS6000_STARTING_FRAME_OFFSET;
}
\f
-/* Internal function to return the built-in function id for the complex
- multiply operation for a given mode. */
-
-static inline built_in_function
-complex_multiply_builtin_code (machine_mode mode)
-{
- gcc_assert (IN_RANGE (mode, MIN_MODE_COMPLEX_FLOAT, MAX_MODE_COMPLEX_FLOAT));
- int func = BUILT_IN_COMPLEX_MUL_MIN + mode - MIN_MODE_COMPLEX_FLOAT;
- return (built_in_function) func;
-}
-
-/* Internal function to return the built-in function id for the complex divide
- operation for a given mode. */
-
-static inline built_in_function
-complex_divide_builtin_code (machine_mode mode)
-{
- gcc_assert (IN_RANGE (mode, MIN_MODE_COMPLEX_FLOAT, MAX_MODE_COMPLEX_FLOAT));
- int func = BUILT_IN_COMPLEX_DIV_MIN + mode - MIN_MODE_COMPLEX_FLOAT;
- return (built_in_function) func;
-}
/* On 64-bit Linux and Freebsd systems, possibly switch the long double library
function names from <foo>l to <foo>f128 if the default long double type is
@@ -28221,53 +28246,11 @@ complex_divide_builtin_code (machine_mode mode)
only do this transformation if the __float128 type is enabled. This
prevents us from doing the transformation on older 32-bit ports that might
have enabled using IEEE 128-bit floating point as the default long double
- type.
-
- We also use the TARGET_MANGLE_DECL_ASSEMBLER_NAME hook to change the
- function names used for complex multiply and divide to the appropriate
- names. */
+ type. */
static tree
rs6000_mangle_decl_assembler_name (tree decl, tree id)
{
- /* Handle complex multiply/divide. For IEEE 128-bit, use __mulkc3 or
- __divkc3 and for IBM 128-bit use __multc3 and __divtc3. */
- if (TARGET_FLOAT128_TYPE
- && TREE_CODE (decl) == FUNCTION_DECL
- && DECL_IS_UNDECLARED_BUILTIN (decl)
- && DECL_BUILT_IN_CLASS (decl) == BUILT_IN_NORMAL)
- {
- built_in_function id = DECL_FUNCTION_CODE (decl);
- const char *newname = NULL;
-
- if (id == complex_multiply_builtin_code (KCmode))
- newname = "__mulkc3";
-
- else if (id == complex_multiply_builtin_code (ICmode))
- newname = "__multc3";
-
- else if (id == complex_multiply_builtin_code (TCmode))
- newname = (TARGET_IEEEQUAD) ? "__mulkc3" : "__multc3";
-
- else if (id == complex_divide_builtin_code (KCmode))
- newname = "__divkc3";
-
- else if (id == complex_divide_builtin_code (ICmode))
- newname = "__divtc3";
-
- else if (id == complex_divide_builtin_code (TCmode))
- newname = (TARGET_IEEEQUAD) ? "__divkc3" : "__divtc3";
-
- if (newname)
- {
- if (TARGET_DEBUG_BUILTIN)
- fprintf (stderr, "Map complex mul/div => %s\n", newname);
-
- return get_identifier (newname);
- }
- }
-
- /* Map long double built-in functions if long double is IEEE 128-bit. */
if (TARGET_FLOAT128_TYPE && TARGET_IEEEQUAD && TARGET_LONG_DOUBLE_128
&& TREE_CODE (decl) == FUNCTION_DECL
&& DECL_IS_UNDECLARED_BUILTIN (decl)
diff --git a/gcc/testsuite/gcc.target/powerpc/divic3-1.c b/gcc/testsuite/gcc.target/powerpc/divic3-1.c
deleted file mode 100644
index 9c9136634f6..00000000000
--- a/gcc/testsuite/gcc.target/powerpc/divic3-1.c
+++ /dev/null
@@ -1,21 +0,0 @@
-/* { dg-do compile } */
-/* { dg-require-effective-target ppc_float128_sw } */
-/* { dg-options "-O2 -mabi=ieeelongdouble -Wno-psabi" } */
-
-/* When GCC is configured with an older library that does not support IEEE
- 128-bit, it issues a warning if you change the long double type. We use
- -Wno-psabi to silence this warning. Since this is a code generation test,
- it does not matter if the library has full IEEE 128-bit support. */
-
-/* Check that complex divide generates the right call for __ibm128 when long
- double is IEEE 128-bit floating point. */
-
-typedef _Complex long double c_ibm128_t __attribute__((mode(__IC__)));
-
-void
-divide (c_ibm128_t *p, c_ibm128_t *q, c_ibm128_t *r)
-{
- *p = *q / *r;
-}
-
-/* { dg-final { scan-assembler "bl __divtc3" } } */
diff --git a/gcc/testsuite/gcc.target/powerpc/divic3-2.c b/gcc/testsuite/gcc.target/powerpc/divic3-2.c
deleted file mode 100644
index bbd10c6986a..00000000000
--- a/gcc/testsuite/gcc.target/powerpc/divic3-2.c
+++ /dev/null
@@ -1,25 +0,0 @@
-/* { dg-do compile } */
-/* { dg-require-effective-target ppc_float128_sw } */
-/* { dg-require-effective-target longdouble128 } */
-/* { dg-options "-O2 -mabi=ibmlongdouble -Wno-psabi" } */
-
-/* When GCC is configured with an older library that does not support IEEE
- 128-bit, it issues a warning if you change the long double type. We use
- -Wno-psabi to silence this warning. Since this is a code generation test,
- it does not matter if the library has full IEEE 128-bit support.
-
- We also need to require that the default long double is 128-bits, otherwise
- the TC/TF modes might not be available. */
-
-/* Check that complex divide generates the right call for __ibm128 when long
- double is IBM 128-bit floating point. */
-
-typedef _Complex long double c_ibm128_t __attribute__((mode(__TC__)));
-
-void
-divide (c_ibm128_t *p, c_ibm128_t *q, c_ibm128_t *r)
-{
- *p = *q / *r;
-}
-
-/* { dg-final { scan-assembler "bl __divtc3" } } */
diff --git a/gcc/testsuite/gcc.target/powerpc/mulic3-1.c b/gcc/testsuite/gcc.target/powerpc/mulic3-1.c
deleted file mode 100644
index d4045177371..00000000000
--- a/gcc/testsuite/gcc.target/powerpc/mulic3-1.c
+++ /dev/null
@@ -1,21 +0,0 @@
-/* { dg-do compile } */
-/* { dg-require-effective-target ppc_float128_sw } */
-/* { dg-options "-O2 -mabi=ieeelongdouble -Wno-psabi" } */
-
-/* When GCC is configured with an older library that does not support IEEE
- 128-bit, it issues a warning if you change the long double type. We use
- -Wno-psabi to silence this warning. Since this is a code generation test,
- it does not matter if the library has full IEEE 128-bit support. */
-
-/* Check that complex multiply generates the right call for __ibm128 when long
- double is IEEE 128-bit floating point. */
-
-typedef _Complex long double c_ibm128_t __attribute__((mode(__IC__)));
-
-void
-multiply (c_ibm128_t *p, c_ibm128_t *q, c_ibm128_t *r)
-{
- *p = *q * *r;
-}
-
-/* { dg-final { scan-assembler "__multc3" } } */
diff --git a/gcc/testsuite/gcc.target/powerpc/mulic3-2.c b/gcc/testsuite/gcc.target/powerpc/mulic3-2.c
deleted file mode 100644
index 97057d3b327..00000000000
--- a/gcc/testsuite/gcc.target/powerpc/mulic3-2.c
+++ /dev/null
@@ -1,25 +0,0 @@
-/* { dg-do compile } */
-/* { dg-require-effective-target ppc_float128_sw } */
-/* { dg-require-effective-target longdouble128 } */
-/* { dg-options "-O2 -mabi=ibmlongdouble -Wno-psabi" } */
-
-/* When GCC is configured with an older library that does not support IEEE
- 128-bit, it issues a warning if you change the long double type. We use
- -Wno-psabi to silence this warning. Since this is a code generation test,
- it does not matter if the library has full IEEE 128-bit support.
-
- We also need to require that the default long double is 128-bits, otherwise
- the TC/TF modes might not be available. */
-
-/* Check that complex multiply generates the right call for __ibm128 when long
- double is IBM 128-bit floating point. */
-
-typedef _Complex long double c_ibm128_t __attribute__((mode(__TC__)));
-
-void
-multiply (c_ibm128_t *p, c_ibm128_t *q, c_ibm128_t *r)
-{
- *p = *q * *r;
-}
-
-/* { dg-final { scan-assembler "__multc3" } } */
^ permalink raw reply [flat|nested] 3+ messages in thread
* [gcc(refs/users/meissner/heads/dmf010)] Revert patches
@ 2023-03-09 5:04 Michael Meissner
0 siblings, 0 replies; 3+ messages in thread
From: Michael Meissner @ 2023-03-09 5:04 UTC (permalink / raw)
To: gcc-cvs
https://gcc.gnu.org/g:d9a94c7caf9ff50ceb9427d1b4c30f8fe53d5124
commit d9a94c7caf9ff50ceb9427d1b4c30f8fe53d5124
Author: Michael Meissner <meissner@linux.ibm.com>
Date: Thu Mar 9 00:04:43 2023 -0500
Revert patches
Diff:
---
gcc/config/rs6000/rs6000.cc | 111 ++++++++++++----------------
gcc/testsuite/gcc.target/powerpc/divic3-1.c | 21 ------
gcc/testsuite/gcc.target/powerpc/divic3-2.c | 25 -------
gcc/testsuite/gcc.target/powerpc/mulic3-1.c | 21 ------
gcc/testsuite/gcc.target/powerpc/mulic3-2.c | 25 -------
5 files changed, 47 insertions(+), 156 deletions(-)
diff --git a/gcc/config/rs6000/rs6000.cc b/gcc/config/rs6000/rs6000.cc
index b1b35b4bb77..8e0b0d022db 100644
--- a/gcc/config/rs6000/rs6000.cc
+++ b/gcc/config/rs6000/rs6000.cc
@@ -11154,6 +11154,26 @@ init_float128_ibm (machine_mode mode)
}
}
+/* Create a decl for either complex long double multiply or complex long double
+ divide when long double is IEEE 128-bit floating point. We can't use
+ __multc3 and __divtc3 because the original long double using IBM extended
+ double used those names. The complex multiply/divide functions are encoded
+ as builtin functions with a complex result and 4 scalar inputs. */
+
+static void
+create_complex_muldiv (const char *name, built_in_function fncode, tree fntype)
+{
+ tree fndecl = add_builtin_function (name, fntype, fncode, BUILT_IN_NORMAL,
+ name, NULL_TREE);
+
+ set_builtin_decl (fncode, fndecl, true);
+
+ if (TARGET_DEBUG_BUILTIN)
+ fprintf (stderr, "create complex %s, fncode: %d\n", name, (int) fncode);
+
+ return;
+}
+
/* Set up IEEE 128-bit floating point routines. Use different names if the
arguments can be passed in a vector register. The historical PowerPC
implementation of IEEE 128-bit floating point used _q_<op> for the names, so
@@ -11165,6 +11185,32 @@ init_float128_ieee (machine_mode mode)
{
if (FLOAT128_VECTOR_P (mode))
{
+ static bool complex_muldiv_init_p = false;
+
+ /* Set up to call __mulkc3 and __divkc3 under -mabi=ieeelongdouble. If
+ we have clone or target attributes, this will be called a second
+ time. We want to create the built-in function only once. */
+ if (mode == TFmode && TARGET_IEEEQUAD && !complex_muldiv_init_p)
+ {
+ complex_muldiv_init_p = true;
+ built_in_function fncode_mul =
+ (built_in_function) (BUILT_IN_COMPLEX_MUL_MIN + TCmode
+ - MIN_MODE_COMPLEX_FLOAT);
+ built_in_function fncode_div =
+ (built_in_function) (BUILT_IN_COMPLEX_DIV_MIN + TCmode
+ - MIN_MODE_COMPLEX_FLOAT);
+
+ tree fntype = build_function_type_list (complex_long_double_type_node,
+ long_double_type_node,
+ long_double_type_node,
+ long_double_type_node,
+ long_double_type_node,
+ NULL_TREE);
+
+ create_complex_muldiv ("__mulkc3", fncode_mul, fntype);
+ create_complex_muldiv ("__divkc3", fncode_div, fntype);
+ }
+
set_optab_libfunc (add_optab, mode, "__addkf3");
set_optab_libfunc (sub_optab, mode, "__subkf3");
set_optab_libfunc (neg_optab, mode, "__negkf2");
@@ -28182,27 +28228,6 @@ rs6000_starting_frame_offset (void)
return RS6000_STARTING_FRAME_OFFSET;
}
\f
-/* Internal function to return the built-in function id for the complex
- multiply operation for a given mode. */
-
-static inline built_in_function
-complex_multiply_builtin_code (machine_mode mode)
-{
- gcc_assert (IN_RANGE (mode, MIN_MODE_COMPLEX_FLOAT, MAX_MODE_COMPLEX_FLOAT));
- return (built_in_function) (BUILT_IN_COMPLEX_MUL_MIN + mode
- - MIN_MODE_COMPLEX_FLOAT);
-}
-
-/* Internal function to return the built-in function id for the complex divide
- operation for a given mode. */
-
-static inline built_in_function
-complex_divide_builtin_code (machine_mode mode)
-{
- gcc_assert (IN_RANGE (mode, MIN_MODE_COMPLEX_FLOAT, MAX_MODE_COMPLEX_FLOAT));
- return (built_in_function) (BUILT_IN_COMPLEX_DIV_MIN + mode
- - MIN_MODE_COMPLEX_FLOAT);
-}
/* On 64-bit Linux and Freebsd systems, possibly switch the long double library
function names from <foo>l to <foo>f128 if the default long double type is
@@ -28221,53 +28246,11 @@ complex_divide_builtin_code (machine_mode mode)
only do this transformation if the __float128 type is enabled. This
prevents us from doing the transformation on older 32-bit ports that might
have enabled using IEEE 128-bit floating point as the default long double
- type.
-
- We also use the TARGET_MANGLE_DECL_ASSEMBLER_NAME hook to change the
- function names used for complex multiply and divide to the appropriate
- names. */
+ type. */
static tree
rs6000_mangle_decl_assembler_name (tree decl, tree id)
{
- /* Handle complex multiply/divide. For IEEE 128-bit, use __mulkc3 or
- __divkc3 and for IBM 128-bit use __multc3 and __divtc3. */
- if (TARGET_FLOAT128_TYPE
- && TREE_CODE (decl) == FUNCTION_DECL
- && DECL_IS_UNDECLARED_BUILTIN (decl)
- && DECL_BUILT_IN_CLASS (decl) == BUILT_IN_NORMAL)
- {
- built_in_function id = DECL_FUNCTION_CODE (decl);
- const char *newname = NULL;
-
- if (id == complex_multiply_builtin_code (KCmode))
- newname = "__mulkc3";
-
- else if (id == complex_multiply_builtin_code (ICmode))
- newname = "__multc3";
-
- else if (id == complex_multiply_builtin_code (TCmode))
- newname = (TARGET_IEEEQUAD) ? "__mulkc3" : "__multc3";
-
- else if (id == complex_divide_builtin_code (KCmode))
- newname = "__divkc3";
-
- else if (id == complex_divide_builtin_code (ICmode))
- newname = "__divtc3";
-
- else if (id == complex_divide_builtin_code (TCmode))
- newname = (TARGET_IEEEQUAD) ? "__divkc3" : "__divtc3";
-
- if (newname)
- {
- if (TARGET_DEBUG_BUILTIN)
- fprintf (stderr, "Map complex mul/div => %s\n", newname);
-
- return get_identifier (newname);
- }
- }
-
- /* Map long double built-in functions if long double is IEEE 128-bit. */
if (TARGET_FLOAT128_TYPE && TARGET_IEEEQUAD && TARGET_LONG_DOUBLE_128
&& TREE_CODE (decl) == FUNCTION_DECL
&& DECL_IS_UNDECLARED_BUILTIN (decl)
diff --git a/gcc/testsuite/gcc.target/powerpc/divic3-1.c b/gcc/testsuite/gcc.target/powerpc/divic3-1.c
deleted file mode 100644
index 9c9136634f6..00000000000
--- a/gcc/testsuite/gcc.target/powerpc/divic3-1.c
+++ /dev/null
@@ -1,21 +0,0 @@
-/* { dg-do compile } */
-/* { dg-require-effective-target ppc_float128_sw } */
-/* { dg-options "-O2 -mabi=ieeelongdouble -Wno-psabi" } */
-
-/* When GCC is configured with an older library that does not support IEEE
- 128-bit, it issues a warning if you change the long double type. We use
- -Wno-psabi to silence this warning. Since this is a code generation test,
- it does not matter if the library has full IEEE 128-bit support. */
-
-/* Check that complex divide generates the right call for __ibm128 when long
- double is IEEE 128-bit floating point. */
-
-typedef _Complex long double c_ibm128_t __attribute__((mode(__IC__)));
-
-void
-divide (c_ibm128_t *p, c_ibm128_t *q, c_ibm128_t *r)
-{
- *p = *q / *r;
-}
-
-/* { dg-final { scan-assembler "bl __divtc3" } } */
diff --git a/gcc/testsuite/gcc.target/powerpc/divic3-2.c b/gcc/testsuite/gcc.target/powerpc/divic3-2.c
deleted file mode 100644
index bbd10c6986a..00000000000
--- a/gcc/testsuite/gcc.target/powerpc/divic3-2.c
+++ /dev/null
@@ -1,25 +0,0 @@
-/* { dg-do compile } */
-/* { dg-require-effective-target ppc_float128_sw } */
-/* { dg-require-effective-target longdouble128 } */
-/* { dg-options "-O2 -mabi=ibmlongdouble -Wno-psabi" } */
-
-/* When GCC is configured with an older library that does not support IEEE
- 128-bit, it issues a warning if you change the long double type. We use
- -Wno-psabi to silence this warning. Since this is a code generation test,
- it does not matter if the library has full IEEE 128-bit support.
-
- We also need to require that the default long double is 128-bits, otherwise
- the TC/TF modes might not be available. */
-
-/* Check that complex divide generates the right call for __ibm128 when long
- double is IBM 128-bit floating point. */
-
-typedef _Complex long double c_ibm128_t __attribute__((mode(__TC__)));
-
-void
-divide (c_ibm128_t *p, c_ibm128_t *q, c_ibm128_t *r)
-{
- *p = *q / *r;
-}
-
-/* { dg-final { scan-assembler "bl __divtc3" } } */
diff --git a/gcc/testsuite/gcc.target/powerpc/mulic3-1.c b/gcc/testsuite/gcc.target/powerpc/mulic3-1.c
deleted file mode 100644
index d4045177371..00000000000
--- a/gcc/testsuite/gcc.target/powerpc/mulic3-1.c
+++ /dev/null
@@ -1,21 +0,0 @@
-/* { dg-do compile } */
-/* { dg-require-effective-target ppc_float128_sw } */
-/* { dg-options "-O2 -mabi=ieeelongdouble -Wno-psabi" } */
-
-/* When GCC is configured with an older library that does not support IEEE
- 128-bit, it issues a warning if you change the long double type. We use
- -Wno-psabi to silence this warning. Since this is a code generation test,
- it does not matter if the library has full IEEE 128-bit support. */
-
-/* Check that complex multiply generates the right call for __ibm128 when long
- double is IEEE 128-bit floating point. */
-
-typedef _Complex long double c_ibm128_t __attribute__((mode(__IC__)));
-
-void
-multiply (c_ibm128_t *p, c_ibm128_t *q, c_ibm128_t *r)
-{
- *p = *q * *r;
-}
-
-/* { dg-final { scan-assembler "__multc3" } } */
diff --git a/gcc/testsuite/gcc.target/powerpc/mulic3-2.c b/gcc/testsuite/gcc.target/powerpc/mulic3-2.c
deleted file mode 100644
index 97057d3b327..00000000000
--- a/gcc/testsuite/gcc.target/powerpc/mulic3-2.c
+++ /dev/null
@@ -1,25 +0,0 @@
-/* { dg-do compile } */
-/* { dg-require-effective-target ppc_float128_sw } */
-/* { dg-require-effective-target longdouble128 } */
-/* { dg-options "-O2 -mabi=ibmlongdouble -Wno-psabi" } */
-
-/* When GCC is configured with an older library that does not support IEEE
- 128-bit, it issues a warning if you change the long double type. We use
- -Wno-psabi to silence this warning. Since this is a code generation test,
- it does not matter if the library has full IEEE 128-bit support.
-
- We also need to require that the default long double is 128-bits, otherwise
- the TC/TF modes might not be available. */
-
-/* Check that complex multiply generates the right call for __ibm128 when long
- double is IBM 128-bit floating point. */
-
-typedef _Complex long double c_ibm128_t __attribute__((mode(__TC__)));
-
-void
-multiply (c_ibm128_t *p, c_ibm128_t *q, c_ibm128_t *r)
-{
- *p = *q * *r;
-}
-
-/* { dg-final { scan-assembler "__multc3" } } */
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-03-09 23:26 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-03-09 16:52 [gcc(refs/users/meissner/heads/dmf010)] Revert patches Michael Meissner
-- strict thread matches above, loose matches on Subject: below --
2023-03-09 23:26 Michael Meissner
2023-03-09 5:04 Michael Meissner
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).