[PATCH] fix for aarch64 sim tbnz bug

Mike Frysinger vapier@gentoo.org
Mon Dec 12 18:42:00 GMT 2016


On 04 Dec 2016 19:43, Jim Wilson wrote:
> --- a/sim/testsuite/sim/aarch64/testutils.inc
> +++ b/sim/testsuite/sim/aarch64/testutils.inc
> @@ -43,11 +43,11 @@
>  	
>  	swiwrite 5
>  	exit 0
> +	.endm
>  
>  	.data
>  .Lpass:
>  	.asciz "pass\n"
> -	.endm
>  
>  # MACRO: fail
>  # Write 'fail' to stdout and quit
> @@ -56,12 +56,12 @@
>  	adrp	x1, .Lfail
>  	add	x1, x1, :lo12:.Lfail
>  	swiwrite 5
> -	exit 0
> +	exit 1
> +	.endm
>  
>  	.data
>  .Lfail:
>  	.asciz "fail\n"
> -	.endm
>  
>  # MACRO: start
>  # All assembler tests should start with a call to "start"

i think you want to stuff those pass/fail strings into the
start macro instead of just having it get inserted where the
include happened to show up.
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://sourceware.org/pipermail/gdb-patches/attachments/20161212/ee3f48f4/attachment.sig>


More information about the Gdb-patches mailing list