Ping.

On 4/26/22 3:06 PM, Pat Haugen via Gcc-patches wrote:
Fix register count when not splitting Complex IEEE 128-bit args.

For ABI_V4, we do not split complex args. This created a problem because
even though an arg would be passed in two VSX regs, we were only advancing the
function arg counter by one VSX register. Fixed with this patch.

Bootstrapped and regression tested on powerpc64(32/64) and powerpc64le.
Ok for master?

-Pat


2022-04-26  Pat Haugen  <pthau...@linux.ibm.com>

     PR testsuite/99685

gcc/
     * config/rs6000/rs6000-call.cc (rs6000_function_arg_advance_1): Bump
     register count when not splitting IEEE 128-bit Complex.


diff --git a/gcc/config/rs6000/rs6000-call.cc b/gcc/config/rs6000/rs6000-call.cc
index f06c692..9d18607 100644
--- a/gcc/config/rs6000/rs6000-call.cc
+++ b/gcc/config/rs6000/rs6000-call.cc
@@ -1111,6 +1111,12 @@ rs6000_function_arg_advance_1 (CUMULATIVE_ARGS *cum, machine_mode mode,
      {
        cum->vregno += n_elts;

+      /* If we are not splitting Complex IEEE 128-bit args then account for
+         the fact that they are passed in 2 VSX regs. */
+      if (! targetm.calls.split_complex_arg && type
+          && TREE_CODE (type) == COMPLEX_TYPE && elt_mode == KCmode)
+        cum->vregno++;
+
        if (!TARGET_ALTIVEC)
          error ("cannot pass argument in vector register because"
             " altivec instructions are disabled, use %qs"

Reply via email to