This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH, binutils, ARM, 0/16] Add support for Armv8.1-M Mainline
- From: Nick Clifton <nickc at redhat dot com>
- To: "Andre Vieira (lists)" <andre dot simoesdiasvieira at arm dot com>, binutils at sourceware dot org
- Date: Thu, 11 Apr 2019 11:04:54 +0100
- Subject: Re: [PATCH, binutils, ARM, 0/16] Add support for Armv8.1-M Mainline
- References: <cfedf9a3-8a0e-d9cd-3b65-683d00e83621@arm.com>
Hi Andre,
> This is a patch series to implement support for Armv8.1-M Mainline. The specifications for the Armv8.1-M Mainline can be found in https://developer.arm.com/docs/ddi0553/latest
The patch series looks fine to me, apart from two small problems:
1) It introduces some assembler testsuite failures for non-ELF based
ARM toolchains:
Checking Binutils in: arm-wince-pe ...
GAS REGRESSION: Valid Armv8.1-M Mainline BF instruction
GAS REGRESSION: Valid Armv8.1-M Mainline BFCSEL instruction
GAS REGRESSION: Valid Armv8.1-M Mainline BFL instruction
GAS REGRESSION: Valid Armv8.1-M Mainline Low Overhead
GAS REGRESSION: Valid Armv8.1-M Mainline BF instruction
GAS REGRESSION: Valid Armv8.1-M Mainline BFCSEL instruction
GAS REGRESSION: Valid Armv8.1-M Mainline BFL instruction
GAS REGRESSION: Valid Armv8.1-M Mainline Low Overhead
GAS REGRESSION: Valid Armv8.1-M Mainline BF instruction
GAS REGRESSION: Valid Armv8.1-M Mainline BFCSEL instruction
GAS REGRESSION: Valid Armv8.1-M Mainline BFL instruction
GAS REGRESSION: Valid Armv8.1-M Mainline Low Overhead
GAS REGRESSION: Valid Armv8.1-M Mainline BF instruction
GAS REGRESSION: Valid Armv8.1-M Mainline BFCSEL instruction
GAS REGRESSION: Valid Armv8.1-M Mainline BFL instruction
GAS REGRESSION: Valid Armv8.1-M Mainline Low Overhead
2) It introduces some assembler and linker testsuite failures for
big-endian ARM toolchains:
Checking Binutils in: armeb-eabi ...
GAS REGRESSION: Valid Armv8.1-M Mainline Low Overhead
GAS REGRESSION: Valid Armv8.1-M Mainline Low Overhead
GAS REGRESSION: Valid Armv8.1-M Mainline Low Overhead
GAS REGRESSION: Valid Armv8.1-M Mainline Low Overhead
LD REGRESSION: Armv8.1-M Mainline BF
LD REGRESSION: Armv8.1-M Mainline BFL
LD REGRESSION: Armv8.1-M Mainline BFCSEL
LD REGRESSION: Armv8.1-M Mainline BF
LD REGRESSION: Armv8.1-M Mainline BFL
LD REGRESSION: Armv8.1-M Mainline BFCSEL
LD REGRESSION: Armv8.1-M Mainline BF
LD REGRESSION: Armv8.1-M Mainline BFL
LD REGRESSION: Armv8.1-M Mainline BFCSEL
LD REGRESSION: Armv8.1-M Mainline BF
LD REGRESSION: Armv8.1-M Mainline BFL
LD REGRESSION: Armv8.1-M Mainline BFCSEL
Would you mind taking a look at these please ?
Cheers
Nick
PS. This was from testing with the entire patch series applied.
I have not narrowed down the specific patches that introduce the
new failures, but I assume that it will be simple for you to do
this yourself.