From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 48) id 216043858D32; Wed, 12 Apr 2023 11:14:29 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 216043858D32 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1681298069; bh=yEkBFlCUISnA2EKZH55bDKHK+k7SYfGxw+ODu3/bfaM=; h=From:To:Subject:Date:In-Reply-To:References:From; b=wvg9hvCqYJoFIObbwlyRfteSW6U6hr1K9B4pncq8glJAuyJpzmWbE2HI8ubZX0D/Z KKtra5nosCU5laZxdsPwQaufeBTU7qh51vnNTKIPE2qedCk/7gjtZOB5GDIsyZIEyA JSy8LX2qZmo8NXNrVS64K0LU8a0USgdRQSAORfXY= From: "rguenth at gcc dot gnu.org" To: gcc-bugs@gcc.gnu.org Subject: [Bug target/109476] Missing optimization for 8bit/8bit multiplication / regression Date: Wed, 12 Apr 2023 11:14:28 +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: missed-optimization X-Bugzilla-Severity: normal X-Bugzilla-Who: rguenth at gcc dot gnu.org 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: cc 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=3D109476 Richard Biener changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |rguenth at gcc dot gnu.org --- Comment #2 from Richard Biener --- I get _Z3mulhj: /* prologue: function */ /* frame size =3D 0 */ /* stack size =3D 0 */ .L__stack_usage =3D 0 mov r22,r24 mov r24,r23 ldi r23,0 ldi r25,0 rcall __mulhi3 but probably because you didn't specify the actual command to compile and I'm using the wrong subarchitecture. What probably regressed is that we're promoting the multiplication to 'int'. The same happens for the + 1 variant for me though. Can you share the -march required to have 16bit multiplication?=