[PATCH][GAS][AArch64] Add new fields and operands required for Armv8.4-a [Patch (1/6)]

Tamar Christina Tamar.Christina@arm.com
Thu Nov 9 11:48:00 GMT 2017


Hi Nick,
> 
> Hi Tamar,
> 
>   I am very sorry, but I have mucked up.  Twice...
> 
> Firstly:
> 
> >> This patch adds the new Fields and Operand types for the new
> >> instructions in Armv8.4-a.
> 
> It was not this patch that triggered the build problem.  It was the patch for
> "Retroactively split CRYPTO into AES and SHA2.Q":
> 
>   https://www.sourceware.org/ml/binutils/2017-11/msg00068.html
> 
> Secondly, I committed the patch in that email, even though it did break 32-bit
> builds.  Doh.  (I should have stayed in bed yesterday).

Yes I noticed this when I was testing the patches, but since it was already committed
I fixed it in the next patch :)

> 
> So - I have checked in a supplemental patch to change the underlying type of
> the aarch64_feature_set typedef, which I think is what you wanted.

Yes I just widend aarch64_feature_set, thanks!

> 
> Also, the patch for the "Add new fields and operands required for Armv8.4-a
> [Patch (1/6)]"
> submission is approved - so please could it be applied.
> 
> Cheers
>   Nick
> 
> include/ChangeLog
> 2017-11-09  Nick Clifton  <nickc@redhat.com>
> 
> 	* opcode/aarch64.h (aarch64_feature_set): Change type to unsigned
> 	long long.
> 
> diff --git a/include/opcode/aarch64.h b/include/opcode/aarch64.h index
> 325b757b1c..a597ebe0cb 100644
> --- a/include/opcode/aarch64.h
> +++ b/include/opcode/aarch64.h
> @@ -82,7 +82,7 @@ typedef uint32_t aarch64_insn;
>  #define AARCH64_ANY		AARCH64_FEATURE (-1, 0)	/*
> Any basic core.  */
> 
>  /* CPU-specific features.  */
> -typedef unsigned long aarch64_feature_set;
> +typedef unsigned long long aarch64_feature_set;
> 
>  #define AARCH64_CPU_HAS_ALL_FEATURES(CPU,FEAT)	\
>    ((~(CPU) & (FEAT)) == 0)


More information about the Binutils mailing list