From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 48) id 60E303858404; Tue, 10 Jan 2023 10:26:03 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 60E303858404 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1673346363; bh=7MstsXzo0cZ0C+HD6DznXrFJCwgM6HXF0SwKaz9FSNI=; h=From:To:Subject:Date:In-Reply-To:References:From; b=oKtPk2pn6FOsHaHgdBqdW5zpdzFYu92ICGkjIDa4x41hGKe4rMlnuSrVQ2hDHimz6 pptAcIBlIWkVAVXRdludFuCbsY4kzt8VbAH16tsC/mp6oyKrNxNjtYI76uVxwSAGCk 7XJz7UGCAjM3iwnYau4mqUGDRkp0hg2sAiyKkyPA= From: "cvs-commit at gcc dot gnu.org" To: gcc-bugs@gcc.gnu.org Subject: [Bug target/108140] ICE expanding __rbit Date: Tue, 10 Jan 2023 10:26:02 +0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: gcc X-Bugzilla-Component: target X-Bugzilla-Version: 12.2.0 X-Bugzilla-Keywords: ice-on-valid-code X-Bugzilla-Severity: normal X-Bugzilla-Who: cvs-commit at gcc dot gnu.org X-Bugzilla-Status: ASSIGNED X-Bugzilla-Resolution: X-Bugzilla-Priority: P3 X-Bugzilla-Assigned-To: ktkachov at gcc dot gnu.org X-Bugzilla-Target-Milestone: 12.3 X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: Message-ID: In-Reply-To: References: 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 List-Id: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=3D108140 --- Comment #7 from CVS Commits --- The releases/gcc-12 branch has been updated by Kyrylo Tkachov : https://gcc.gnu.org/g:849c3cf7b4189342b4a0df941afddf8327585570 commit r12-9037-g849c3cf7b4189342b4a0df941afddf8327585570 Author: Kyrylo Tkachov Date: Mon Dec 19 11:16:47 2022 +0000 aarch64: PR target/108140 Handle NULL target in data intrinsic expansion In this PR we ICE when expanding the __rbit builtin with a NULL target = rtx. I *think* that only happens when the result is unused and hence maybe we shouldn't be expanding any RTL at all, but the ICE here is easily fixed by deriving the mode f= rom the type of the expression rather than the target. This patch does that. Bootstrapped and tested on aarch64-none-linux-gnu. gcc/ChangeLog: PR target/108140 * config/aarch64/aarch64-builtins.cc (aarch64_expand_builtin_data_intrinsic): Handle NULL target. gcc/testsuite/ChangeLog: PR target/108140 * gcc.target/aarch64/acle/pr108140.c: New test. (cherry picked from commit 98756bcbe27647f263f2b312d1d933d70cf56ba9)=