Opened 3 years ago

Closed 3 years ago

#15350 closed enhancement (fixed)

Patch GCC to fix PR101749

Reported by: Xi Ruoyao Owned by: Bruce Dubbs
Priority: normal Milestone: 11.0
Component: BOOK Version: git
Severity: normal Keywords:
Cc:

Description

With Glibc-2.34, we are getting some new gcc test failures. Among them PR101749 is a real issue, breaking "gcc -fsanitize=address -static-libasan" totally:

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

A patch has been applied for gcc-11.x upstream, we can apply it for gcc.

Change History (8)

comment:1 by ken@…, 3 years ago

I think this is needed in LFS as well ?

in reply to:  1 ; comment:2 by Douglas R. Reno, 3 years ago

Replying to ken@…:

I think this is needed in LFS as well ?

I concur, I think it is needed over there too

comment:3 by Bruce Dubbs, 3 years ago

sed -e '/static.*SIGSTKSZ/d' \
    -e 's/return kAltStackSize/return SIGSTKSZ * 4/' \
    -i libsanitizer/sanitizer_common/sanitizer_posix_libcdep.cpp

Should do it. It applies correctly, but I've not yet rebuilt/tested gcc.

comment:4 by Bruce Dubbs, 3 years ago

Milestone: 10.211.0

Milestone renamed

comment:5 by Bruce Dubbs, 3 years ago

I ran the tests on a current development build and with the above sed. These are the results:

      === g++ tests ===


Running target unix
FAIL: g++.dg/asan/asan_test.C   -O2  (test for excess errors)
UNRESOLVED: g++.dg/asan/asan_test.C   -O2  compilation failed to produce executable

      === g++ Summary ===

# of expected passes    207329
# of unexpected failures   1
# of expected failures     1073
# of unresolved testcases  1
# of unsupported tests     9048
/sources/gcc-11.2.0/build/gcc/xg++  version 11.2.0 (GCC)

      === gcc tests ===


Running target unix
FAIL: gcc.dg/analyzer/analyzer-verbosity-2a.c (test for excess errors)
FAIL: gcc.dg/analyzer/analyzer-verbosity-3a.c (test for excess errors)
FAIL: gcc.dg/analyzer/edges-1.c (test for excess errors)
FAIL: gcc.dg/analyzer/file-1.c (test for excess errors)
FAIL: gcc.dg/analyzer/file-2.c (test for excess errors)
FAIL: gcc.dg/analyzer/file-paths-1.c (test for excess errors)
FAIL: gcc.dg/analyzer/file-pr58237.c (test for excess errors)
FAIL: gcc.dg/analyzer/pr99716-1.c (test for excess errors)

      === gcc Summary ===

# of expected passes    160265
# of unexpected failures   8
# of expected failures     854
# of unsupported tests     2393
/sources/gcc-11.2.0/build/gcc/xgcc  version 11.2.0 (GCC)

Running target unix
FAIL: 17_intro/headers/c++1998/49745.cc (test for excess errors)
FAIL: 22_locale/numpunct/members/char/3.cc execution test
FAIL: 22_locale/time_get/get_time/char/2.cc execution test
FAIL: 22_locale/time_get/get_time/char/wrapped_env.cc execution test
FAIL: 22_locale/time_get/get_time/char/wrapped_locale.cc execution test
FAIL: 22_locale/time_get/get_time/wchar_t/2.cc execution test
FAIL: 22_locale/time_get/get_time/wchar_t/wrapped_env.cc execution test
FAIL: 22_locale/time_get/get_time/wchar_t/wrapped_locale.cc execution test

      === libstdc++ Summary ===

# of expected passes    14821
# of unexpected failures   8
# of expected failures     104
# of unsupported tests     353

All other tests passed.

Version 0, edited 3 years ago by Bruce Dubbs (next)

in reply to:  2 comment:6 by Xi Ruoyao, 3 years ago

Replying to Douglas R. Reno:

Replying to ken@…:

I think this is needed in LFS as well ?

I concur, I think it is needed over there too

Added to LFS r10.1-175. Logs at https://www.linuxfromscratch.org/~xry111/lfs-r10.1-175-logs/.

comment:7 by Bruce Dubbs, 3 years ago

Owner: changed from blfs-book to Bruce Dubbs
Status: newassigned

comment:8 by Bruce Dubbs, 3 years ago

Resolution: fixed
Status: assignedclosed

Fixed at commit 9654a22ae2a4d98cfa44ec06a648117220e0289f

Package updates and a fix for GCC
    Update to libqmi-1.30.0.
    Update to nss-3.69.
    Fix an issue when building gcc identified upstream.
Note: See TracTickets for help on using tickets.