From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <sourceware-bugzilla@sourceware.org>
Received: by sourceware.org (Postfix, from userid 48)
	id DC7BF385841D; Thu, 19 Jan 2023 13:05:12 +0000 (GMT)
DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org DC7BF385841D
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org;
	s=default; t=1674133512;
	bh=UAPAMiyf0h3ZwDSoXmJGtP59pskpJDQydCnqcWvPLjM=;
	h=From:To:Subject:Date:In-Reply-To:References:From;
	b=aIpZ6ewNvTrwdyEgivzLXA780A9IDWdLCH2FYmwf3qCpsm1F32lHAxaP+WEZ96CFo
	 4LBx6l42G0zpKbNdZq9lWxxF7zBCwBjDdMqsx6iB8rPYhbEegYcKUMk3HACfk9Xerp
	 IV6RGoqZhDHPD0eo1/PQ4bsDVjAOXzLAmbzznotA=
From: "cvs-commit at gcc dot gnu.org" <sourceware-bugzilla@sourceware.org>
To: gdb-prs@sourceware.org
Subject: [Bug tdep/30012] [gdb/tdep, aarch64] fp and sp register aliases
 ignored
Date: Thu, 19 Jan 2023 13:05:12 +0000
X-Bugzilla-Reason: CC
X-Bugzilla-Type: changed
X-Bugzilla-Watch-Reason: None
X-Bugzilla-Product: gdb
X-Bugzilla-Component: tdep
X-Bugzilla-Version: HEAD
X-Bugzilla-Keywords: 
X-Bugzilla-Severity: normal
X-Bugzilla-Who: cvs-commit at gcc dot gnu.org
X-Bugzilla-Status: NEW
X-Bugzilla-Resolution: 
X-Bugzilla-Priority: P2
X-Bugzilla-Assigned-To: unassigned at sourceware dot org
X-Bugzilla-Target-Milestone: ---
X-Bugzilla-Flags: 
X-Bugzilla-Changed-Fields: 
Message-ID: <bug-30012-4717-6Cri0pZxoa@http.sourceware.org/bugzilla/>
In-Reply-To: <bug-30012-4717@http.sourceware.org/bugzilla/>
References: <bug-30012-4717@http.sourceware.org/bugzilla/>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
X-Bugzilla-URL: http://sourceware.org/bugzilla/
Auto-Submitted: auto-generated
MIME-Version: 1.0
List-Id: <gdb-prs.sourceware.org>

https://sourceware.org/bugzilla/show_bug.cgi?id=3D30012

--- Comment #2 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gn=
u.org> ---
The master branch has been updated by Tom de Vries <vries@sourceware.org>:

https://sourceware.org/git/gitweb.cgi?p=3Dbinutils-gdb.git;h=3Dea2f6fad7e10=
4fb65da6a438bb0ec753210f1e90

commit ea2f6fad7e104fb65da6a438bb0ec753210f1e90
Author: Tom de Vries <tdevries@space.suse.cz>
Date:   Thu Jan 19 14:05:08 2023 +0100

    [gdb/tdep, aarch64] Remove fp and sp reg aliases, add x31 reg alias

    In aarch64-tdep.c we find these register aliases:
    ...
    {
      /* 64-bit register names.  */
      {"fp", AARCH64_FP_REGNUM},
      {"lr", AARCH64_LR_REGNUM},
      {"sp", AARCH64_SP_REGNUM},
    ...

    The sp alias is superfluous, because the canonical name of x31 is alrea=
dy
sp.

    The fp alias is superfluous, because it's already taken by the default
meaning
    of fp, assigned here in _initialize_frame_reg:
    ...
      user_reg_add_builtin ("fp", value_of_builtin_frame_fp_reg, NULL);
    ...

    Fix this by removing the fp and sp aliases.

    While we're at it, add an x31 alias for sp.

    Approved-By: Luis Machado <luis.machado@arm.com>

    Tested on aarch64-linux.
    PR tdep/30012
    Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=3D30012

--=20
You are receiving this mail because:
You are on the CC list for the bug.=