From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 48) id 83E553842422; Fri, 19 Feb 2021 04:29:22 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 83E553842422 From: "zhan3299 at purdue dot edu" To: gcc-bugs@gcc.gnu.org Subject: [Bug c/99156] New: __builtin_expect affects the interpretation of its first operand Date: Fri, 19 Feb 2021 04:29:22 +0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: new X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: gcc X-Bugzilla-Component: c X-Bugzilla-Version: 11.0 X-Bugzilla-Keywords: X-Bugzilla-Severity: normal X-Bugzilla-Who: zhan3299 at purdue dot edu X-Bugzilla-Status: UNCONFIRMED X-Bugzilla-Resolution: X-Bugzilla-Priority: P3 X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org X-Bugzilla-Target-Milestone: --- X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: bug_id short_desc product version bug_status bug_severity priority component assigned_to reporter target_milestone Message-ID: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/ Auto-Submitted: auto-generated MIME-Version: 1.0 X-BeenThere: gcc-bugs@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gcc-bugs mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 19 Feb 2021 04:29:22 -0000 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=3D99156 Bug ID: 99156 Summary: __builtin_expect affects the interpretation of its first operand Product: gcc Version: 11.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component: c Assignee: unassigned at gcc dot gnu.org Reporter: zhan3299 at purdue dot edu Target Milestone: --- I hope it does not bother. I try to refer to a bug in llvm which may also affect gcc.=20=20 Following are copied-and-pasted from the discussion about a similar bug in clang (https://bugs.llvm.org/show_bug.cgi?id=3D49239#c3).=20 Specifically,=20 > int maybe_vla(int n) { > goto label; > int arr[({0;})]; > label: > return sizeof(arr); > } >=20 > ... is rejected by both Clang and GCC because the statement-expression is= not an ICE, but >=20 > int maybe_vla(int n) { > goto label; > int arr[__builtin_expect(({0;}), 0)]; > label: > return sizeof(arr); > } >=20 > ... is accepted. This seems like a bug in both compilers to me: __builtin= _expect isn't supposed to affect the interpretation of its first operand, a= nd presumably shouldn't be weakening the strict ICE checks. case 1: https://godbolt.org/z/zWGEfx case 2: https://godbolt.org/z/bejfcc=