aarch64: Fix bogus type punning in parse_barrier() [PR26699]
authorAlex Coplan <alex.coplan@arm.com>
Tue, 6 Oct 2020 14:56:44 +0000 (15:56 +0100)
committerAlex Coplan <alex.coplan@arm.com>
Tue, 6 Oct 2020 14:56:44 +0000 (15:56 +0100)
commit05cfb0d8cc9b7f8676f5ae55a93642f091d5405f
treec85ab3d18c56b8ae2d8057ae091c36c774748cbf
parent0ec992e6647c00fc234cdf4c11732b6711b024af
aarch64: Fix bogus type punning in parse_barrier() [PR26699]

This patch fixes a bogus use of type punning in parse_barrier() which
was causing an assembly failure on big endian LP64 hosts when attempting
to assemble "isb sy" for AArch64.

The type of the entries in aarch64_barrier_opt_hsh is
aarch64_name_value_pair. We were incorrectly casting this to the
locally-defined asm_barrier_opt which has a wider type (on LP64) for the
second member. This happened to work on little-endian hosts but fails on
LP64 big endian.

The fix is to use the correct type in parse_barrier(). This makes the
locally-defined asm_barrier_opt redundant, so remove it.

gas/ChangeLog:

* config/tc-aarch64.c (asm_barrier_opt): Delete.
(parse_barrier): Fix bogus type punning.
* testsuite/gas/aarch64/system.d: Update disassembly.
* testsuite/gas/aarch64/system.s: Add isb sy test.
gas/ChangeLog
gas/config/tc-aarch64.c
gas/testsuite/gas/aarch64/system.d
gas/testsuite/gas/aarch64/system.s