From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 48) id C129E384FE9C; Mon, 19 Dec 2022 11:19:52 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org C129E384FE9C DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1671448792; bh=saXVO/J1yPNhqqTqC9ftae2Evshm6x8JWMtGCn9/+YI=; h=From:To:Subject:Date:In-Reply-To:References:From; b=U9zpNolyrtk83cqAm9+MkW3lGuD0F7Pav3U7ZRdcQdy97ssUfe1SHLTEEzZSm0AQ6 sQ3yi7lof4FnPfjBToWxgu9Z1UKhAUN7DioT499bvHgIbs8dm7Mm1b44epWdjEz+9r 1i6ebb7XT82EphW0jEiIj6Bpov+D3GeWuIGkdZs0= From: "cvs-commit at gcc dot gnu.org" To: gcc-bugs@gcc.gnu.org Subject: [Bug target/108140] ICE expanding __rbit Date: Mon, 19 Dec 2022 11:19:51 +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 #6 from CVS Commits --- The master branch has been updated by Kyrylo Tkachov : https://gcc.gnu.org/g:98756bcbe27647f263f2b312d1d933d70cf56ba9 commit r13-4777-g98756bcbe27647f263f2b312d1d933d70cf56ba9 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.=