[aarch64] Remove handling of ADR/ADRP from prologue analyzer
authorLuis Machado <luis.machado@arm.com>
Tue, 16 Aug 2022 08:08:44 +0000 (09:08 +0100)
committerLuis Machado <luis.machado@arm.com>
Thu, 18 Aug 2022 15:42:50 +0000 (16:42 +0100)
commit992545e7df523cb923eeeb21a753778a95da420e
treea2addbc266faaa8061ad3e81190fb2da2e5ae6f7
parent243cf0f69c36c4ee09c3c2b0bc7a97dc16119c51
[aarch64] Remove handling of ADR/ADRP from prologue analyzer

As reported by Tom in https://sourceware.org/pipermail/gdb-patches/2022-August/191357.html,
the aarch64 prologue analyzer considers the adrp instruction in the
gdb.dwarf2/dw2-dir-file-name.exp testcase to be part of a prologue.

The function has no prologue though, and it only loads the volatile variable
from memory.  GDB should not skip any instructions in this case.

Doing some archaeology, it seems handling for adr/adrp in prologues was
included with the original aarch64 port.  It might've been an oversight.

In the particular case of gdb.dwarf2/dw2-dir-file-name.exp, the analyzer skips
a couple instructions and leaves us in a nice spot where the address to the
variable "v" is already in w0. But no prologues exists.

Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=29481
gdb/aarch64-tdep.c