[Bug testsuite/114744] New: test case gcc.target/powerpc/builtins-6-p9-runnable.c fails

[Bug testsuite/114744] New: test case gcc.target/powerpc/builtins-6-p9-runnable.c fails

[seurer at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114744

            Bug ID: 114744
           Summary: test case gcc.target/powerpc/builtins-6-p9-runnable.c
                    fails
           Product: gcc
           Version: 14.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: testsuite
          Assignee: unassigned at gcc dot gnu.org
          Reporter: seurer at gcc dot gnu.org
  Target Milestone: ---

gcc.target/powerpc/builtins-6-p9-runnable.c fails only on power 9 BE for gcc
12, 13, and trunk (14).

FAIL: gcc.target/powerpc/builtins-6-p9-runnable.c  -O0  execution test
FAIL: gcc.target/powerpc/builtins-6-p9-runnable.c  -O1  execution test
FAIL: gcc.target/powerpc/builtins-6-p9-runnable.c  -O2  execution test
FAIL: gcc.target/powerpc/builtins-6-p9-runnable.c  -O3  execution test
FAIL: gcc.target/powerpc/builtins-6-p9-runnable.c  -Os  execution test
FAIL: gcc.target/powerpc/builtins-6-p9-runnable.c execution test

(gdb) run 
Starting program: /home/seurer/gcc/git/build/gcc-12/builtins-6-p9-runnable.exe 
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib/powerpc64-linux-gnu/libthread_db.so.1".

Program received signal SIGABRT, Aborted.
__pthread_kill_implementation (threadid=<optimized out>, signo=<optimized out>,
no_tid=<optimized out>) at pthread_kill.c:44
44      pthread_kill.c: No such file or directory.
(gdb) where
#0  __pthread_kill_implementation (threadid=<optimized out>, signo=<optimized
out>, no_tid=<optimized out>) at pthread_kill.c:44
#1  0x00007ffff7c90bb4 in __GI_raise (sig=<optimized out>) at
../sysdeps/posix/raise.c:26
#2  0x00007ffff7c7490c in __GI_abort () at abort.c:79
#3  0x00000000100006e8 in main () at
/home/seurer/gcc/git/gcc-12/gcc/testsuite/gcc.target/powerpc/builtins-6-p9-runnable.c:434



  vec_xst_len (store_data_sc, address_sc, len);

  if (result_wrong_sc (vec_sc_expected1, vec_sc_result1))
    {
#ifdef DEBUG
       printf("Error: vec_xst_len, len = %d, sc result does not match expected
result\n", len);
       print_sc (vec_sc_expected1, vec_sc_result1);
#else
       abort();  // line 434
#endif
    }




Compiling it with -DDEBUG:

seurer@nilram:~/gcc/git/build/gcc-12$ ./builtins-6-p9-runnable.exe 
Error: vec_xl_len(), len = 8, vec_s128_result1[0] = 0 0; vec_s128_expected1[0]
= 0 0
Error: vec_xl_len(), len = 8, vec_u128_result1[0] = 0; vec_u128_expected1[0] =
0
Error: pre/post initialzation vec_xl_len_r(), len = 16, vec_uc_result1[0] = 1;
vec_uc_expected1[0] = 16
Error: pre/post initialzation vec_xl_len_r(), len = 16, vec_uc_result1[1] = 2;
vec_uc_expected1[1] = 15
Error: pre/post initialzation vec_xl_len_r(), len = 16, vec_uc_result1[2] = 3;
vec_uc_expected1[2] = 14
Error: pre/post initialzation vec_xl_len_r(), len = 16, vec_uc_result1[3] = 4;
vec_uc_expected1[3] = 13
Error: pre/post initialzation vec_xl_len_r(), len = 16, vec_uc_result1[4] = 5;
vec_uc_expected1[4] = 12
Error: pre/post initialzation vec_xl_len_r(), len = 16, vec_uc_result1[5] = 6;
vec_uc_expected1[5] = 11
Error: pre/post initialzation vec_xl_len_r(), len = 16, vec_uc_result1[6] = 7;
vec_uc_expected1[6] = 10
Error: pre/post initialzation vec_xl_len_r(), len = 16, vec_uc_result1[7] = 8;
vec_uc_expected1[7] = 9
Error: pre/post initialzation vec_xl_len_r(), len = 16, vec_uc_result1[8] = 9;
vec_uc_expected1[8] = 8
Error: pre/post initialzation vec_xl_len_r(), len = 16, vec_uc_result1[9] = 10;
vec_uc_expected1[9] = 7
Error: pre/post initialzation vec_xl_len_r(), len = 16, vec_uc_result1[10] =
11; vec_uc_expected1[10] = 6
Error: pre/post initialzation vec_xl_len_r(), len = 16, vec_uc_result1[11] =
12; vec_uc_expected1[11] = 5
Error: pre/post initialzation vec_xl_len_r(), len = 16, vec_uc_result1[12] =
13; vec_uc_expected1[12] = 4
Error: pre/post initialzation vec_xl_len_r(), len = 16, vec_uc_result1[13] =
14; vec_uc_expected1[13] = 3
Error: pre/post initialzation vec_xl_len_r(), len = 16, vec_uc_result1[14] =
15; vec_uc_expected1[14] = 2
Error: pre/post initialzation vec_xl_len_r(), len = 16, vec_uc_result1[15] =
16; vec_uc_expected1[15] = 1

[Bug testsuite/114744] test case gcc.target/powerpc/builtins-6-p9-runnable.c fails

[seurer at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114744

--- Comment #1 from seurer at gcc dot gnu.org ---
also gcc 11

[Bug testsuite/114744] test case gcc.target/powerpc/builtins-6-p9-runnable.c fails

[linkw at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114744

Kewen Lin <linkw at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |linkw at gcc dot gnu.org
     Ever confirmed|0                           |1
   Last reconfirmed|                            |2024-04-17
           Assignee|unassigned at gcc dot gnu.org      |linkw at gcc dot gnu.org
             Status|UNCONFIRMED                 |ASSIGNED

--- Comment #2 from Kewen Lin <linkw at gcc dot gnu.org> ---
This is very very likely a test issue, due to endianness which the load vector
should consider. I'll have a look.

[Bug testsuite/114744] test case gcc.target/powerpc/builtins-6-p9-runnable.c fails

[cvs-commit at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114744

--- Comment #3 from GCC Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Kewen Lin <linkw@gcc.gnu.org>:

https://gcc.gnu.org/g:6e62ede7aaccc6ebe027c8e00224f65e226072e9

commit r14-10011-g6e62ede7aaccc6ebe027c8e00224f65e226072e9
Author: Kewen Lin <linkw@linux.ibm.com>
Date:   Wed Apr 17 22:20:07 2024 -0500

    testsuite, rs6000: Fix builtins-6-p9-runnable.c for BE [PR114744]

    Test case builtins-6-p9-runnable.c doesn't work well on BE
    due to two problems:
      - When applying vec_xl_len onto data_128 and data_u128
        with length 8, it expects to load 1280000[01] from
        the memory, but unfortunately assigning 1280000[01] to
        a {vector} {u,}int128 type variable, the value isn't
        guaranteed to be at the beginning of storage (in the
        low part of memory), which means the loaded value can
        be unexpected (as shown on BE).  So this patch is to
        introduce getU128 which can ensure the given value
        shows up as expected and also update some dumping code
        for debugging.
      - When applying vec_xl_len_r with length 16, on BE it's
        just like the normal vector load, so the expected data
        should not be reversed from the original.

            PR testsuite/114744

    gcc/testsuite/ChangeLog:

            * gcc.target/powerpc/builtins-6-p9-runnable.c: Adjust for BE by
fixing
            data_{u,}128, their uses and vec_uc_expected1, also adjust some
formats.

[Bug testsuite/114744] test case gcc.target/powerpc/builtins-6-p9-runnable.c fails

[linkw at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114744

Kewen Lin <linkw at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |14.0
             Status|ASSIGNED                    |RESOLVED
         Resolution|---                         |FIXED

--- Comment #4 from Kewen Lin <linkw at gcc dot gnu.org> ---
Should be fixed on trunk, since it's a test issue, no backporting need.