[Bug testsuite/96519] New: [11 regression] new test case gcc.dg/ia64-sync-5.c fails

[Bug testsuite/96519] New: [11 regression] new test case gcc.dg/ia64-sync-5.c fails

[seurer at gcc dot gnu.org]

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

            Bug ID: 96519
           Summary: [11 regression] new test case gcc.dg/ia64-sync-5.c
                    fails
           Product: gcc
           Version: 11.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: ---

g:6339f93279b24b5a61eeed2a5040b8b2a0d3ab42, r11-2605 

spawn -ignore SIGHUP /home/seurer/gcc/git/build/gcc-test/gcc/xgcc
-B/home/seurer/gcc/git/build/gcc-test/gcc/
/home/seurer/gcc/git/gcc-test/gcc/testsuite/gcc.dg/ia64-sync-5.c
-fno-diagnostics-show-caret -fno-diagnostics-show-line-numbers
-fdiagnostics-color=never -fdiagnostics-urls=never -ansi -pedantic-errors -lm
-o ./ia64-sync-5.exe
PASS: gcc.dg/ia64-sync-5.c (test for excess errors)
. . .
Execution timeout is: 300
spawn [open ...]
FAIL: gcc.dg/ia64-sync-5.c execution test
testcase /home/seurer/gcc/git/gcc-test/gcc/testsuite/gcc.dg/dg.exp completed in
0 seconds

                === gcc Summary ===

# of expected passes            1
# of unexpected failures        1


(gdb) run
Starting program: /home/seurer/gcc/git/build/gcc-test/./ia64-sync-5.exe 

Program received signal SIGABRT, Aborted.
0x00003fffb7cd258c in __GI_raise (sig=<optimized out>) at
../nptl/sysdeps/unix/sysv/linux/raise.c:55
55        return INLINE_SYSCALL (tgkill, 3, pid, selftid, sig);
(gdb) where
#0  0x00003fffb7cd258c in __GI_raise (sig=<optimized out>) at
../nptl/sysdeps/unix/sysv/linux/raise.c:55
#1  0x00003fffb7cd4778 in __GI_abort () at abort.c:90
#2  0x00000000100006f4 in do_qi () at
/home/seurer/gcc/git/gcc-test/gcc/testsuite/gcc.dg/ia64-sync-5.c:25
#3  0x0000000010000a44 in main () at
/home/seurer/gcc/git/gcc-test/gcc/testsuite/gcc.dg/ia64-sync-5.c:74



static void
do_qi (void)
{ 
  if (__sync_val_compare_and_swap(AC+0, -30, -115) != -30)
    abort ();   // line 25
...

[Bug testsuite/96519] [11 regression] new test case gcc.dg/ia64-sync-5.c fails

[clyon at gcc dot gnu.org]

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

Christophe Lyon <clyon at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Target|powerpc64*-linux-gnu        |powerpc64*-linux-gnu
                   |                            |aarch64 arm
                 CC|                            |clyon at gcc dot gnu.org

--- Comment #1 from Christophe Lyon <clyon at gcc dot gnu.org> ---
Seen also on aarch64 and arm

[Bug testsuite/96519] [11 regression] new test case gcc.dg/ia64-sync-5.c fails

[cvs-commit at gcc dot gnu.org]

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

--- Comment #2 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Kwok Yeung <kcy@gcc.gnu.org>:

https://gcc.gnu.org/g:36507a2c9cbac673553b043aa1817e674836c232

commit r11-2670-g36507a2c9cbac673553b043aa1817e674836c232
Author: Kwok Cheung Yeung <kcy@codesourcery.com>
Date:   Wed Aug 12 05:19:11 2020 -0700

    Fix gcc.dg/ia64-sync-5.c for architectures with unsigned char as default
(PR 96519)

    If char is unsigned, then comparisons of the char array elements against
    negative integers in the test will fail as values in the array will always
    be positive, and will remain so when promoted to signed int.

    2020-08-12  Kwok Cheung Yeung  <kcy@codesourcery.com>

            PR testsuite/96519

            gcc/testsuite/
            * gcc.dg/ia64-sync-5.c (AC, init_qi, test_qi): Change element type
to
            signed char.

[Bug testsuite/96519] [11 regression] new test case gcc.dg/ia64-sync-5.c fails

[rguenth at gcc dot gnu.org]

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

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |11.0

[Bug testsuite/96519] [11 regression] new test case gcc.dg/ia64-sync-5.c fails

[tschwinge at gcc dot gnu.org]

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

Thomas Schwinge <tschwinge at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Assignee|unassigned at gcc dot gnu.org      |kcy at codesourcery dot com
                 CC|                            |tschwinge at gcc dot gnu.org
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |FIXED

--- Comment #3 from Thomas Schwinge <tschwinge at gcc dot gnu.org> ---
As far as I can tell, fixed with Kwok's patch to use explicit 'signed char'.