[PATCH v2 2/5] gas: aarch64: Add instructions for GICv5

Alice Carlotti alice.carlotti@arm.com
Mon Oct 6 15:30:33 GMT 2025


On Mon, Oct 06, 2025 at 03:40:04PM +0100, saurabh.jha@arm.com wrote:
> From: Saurabh Jha <saurabh.jha@arm.com>
> 
> Add new instructions from the Generic Interrupt Controller, GICv5,
> extension. These instructions are aliases to system instructions and are
> the following:
> 
> * gic <operation>, <reg>
> * gicr <reg>, <operation>
> * gsb <operation>
...
> diff --git a/gas/testsuite/gas/aarch64/gcie.s b/gas/testsuite/gas/aarch64/gcie.s
> new file mode 100644
> index 00000000000..d7eb8e1721b
> --- /dev/null
> +++ b/gas/testsuite/gas/aarch64/gcie.s
> @@ -0,0 +1,113 @@
...
> +
> +	// GSB SYS
> +	gsb	sys
> +
> +	// GSB ACK
> +	gsb	ack

Could you add tests for invalid gsb variants, i.e:

+ 	// Don't disassemble gsb with nonzero Rt.
+ 	sys #0, c12, c0, #0, x0
+ 	sys #0, c12, c0, #1, x0

(and matching changes to the .d file, of course).

Also, one more nit - patches 2-5 affect more than just gas, so it would make
sense to drop "gas: " from your commit title.

The series is ok with these changes, thanks.

Alice


More information about the Binutils mailing list