bug#21513: (now an infinite loop in grep/tests/triple-backref?)

2020-01-20 Thread Martin Liška
On 1/18/20 11:16 PM, Paul Eggert wrote: On 1/18/20 6:44 AM, Martin Liška wrote: I can confirm that it never finishes on i586. I can't reproduce the problem, either on Ubuntu 18.04.3 x86-64 (GCC Ubuntu 7.4.0-1ubuntu1~18.04.1) or on Fedora 31 (GCC 9.2.1 20190827 x86-64 (Red Hat 9.2.1-1)). I di

bug#21513: (now an infinite loop in grep/tests/triple-backref?)

2020-01-20 Thread Martin Liška
On 1/20/20 12:06 PM, Andreas Schwab wrote: That appears to be a GCC bug with -fprofile-generate. Or a co-incidence of usage of __builtin_unreachable that can lead to any code optimization (segfault, infinite loop, ...). Martin Andreas.

bug#21513: (now an infinite loop in grep/tests/triple-backref?)

2020-01-20 Thread Andreas Schwab
That appears to be a GCC bug with -fprofile-generate. Andreas. -- Andreas Schwab, SUSE Labs, sch...@suse.de GPG Key fingerprint = 0196 BAD8 1CE9 1970 F4BE 1748 E4D4 88E3 0EEA B9D7 "And now for something completely different."

bug#39206: lib/unistr/u8-uctomb.c fails to compile

2020-01-20 Thread Andreas Schwab
../../grep/lib/unistr/u8-uctomb.c: In function 'u8_uctomb': ../../grep/lib/unistr/u8-uctomb.c:64:65: error: this statement may fall through [-Werror=implicit-fallthrough=] case 4: s[3] = 0x80 | (uc & 0x3f); uc = uc >> 6; uc |= 0x1;

bug#39206: warning in lib/unistr/u8-uctomb.c

2020-01-20 Thread Bruno Haible
Correcting the subject. > cc1: all warnings being treated as errors "this statement may fall through" is a warning. *You* turned it into an error by using -Werror or -Werror=implicit-fallthrough. It is therefore misleading to say that there is a compilation error in lib/unistr/u8-uctomb.c. > ../

bug#21513: (now an infinite loop in grep/tests/triple-backref?)

2020-01-20 Thread Paul Eggert
On 1/20/20 2:49 AM, Martin Liška wrote: In this case an infinite loop is highly undesirable. Sure, but the regex code is littered with highly undesirable code like this, as it's easy to use a weird regexp to make the code explode exponentially and there's little practical difference between