> -----Original Message-----
> From: Lu, Hongjiu
> Sent: Monday, April 23, 2018 1:56 PM
> To: gccpat...@intel.com
> Cc: Tsimbalist, Igor V <igor.v.tsimbal...@intel.com>; Uros Bizjak
> <ubiz...@gmail.com>
> Subject: [PATCH] x86/cet: Use unsigned integer to unwind shadow stack
> 
> Use unsigned integer to unwind shadow stack by replacing jle with jbe and
> jg with ja.
> 
> OK for trunk?

OK.

Igor

> H.J.
> ----
>       PR target/85489
>       * config/x86/sjlj.S (GTM_longjmp): Replace jle/jg with jbe/ja.
> ---
>  libitm/config/x86/sjlj.S | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/libitm/config/x86/sjlj.S b/libitm/config/x86/sjlj.S
> index 3cac9099c9e..e5cdbab48a7 100644
> --- a/libitm/config/x86/sjlj.S
> +++ b/libitm/config/x86/sjlj.S
> @@ -235,7 +235,7 @@ SYM(GTM_longjmp):
>       /* If # of frames is greater 255 then loop
>          and adjust.  */
>       cmpq    $255, %rcx
> -     jle     .L3
> +     jbe     .L3
>       movl    $255, %edi
>       .p2align 4,,10
>       .p2align 3
> @@ -243,7 +243,7 @@ SYM(GTM_longjmp):
>       incsspq %rdi
>       subq    $255, %rcx
>       cmpq    $255, %rcx
> -     jg      .L4
> +     ja      .L4
>  .L3:
>       incsspq %rcx
>  .L1:
> @@ -273,7 +273,7 @@ SYM(GTM_longjmp):
>       /* If # of frames is greater 255 then loop
>          and adjust.  */
>       cmpl    $255, %ecx
> -     jle     .L3
> +     jbe     .L3
>       pushl   %eax
>       movl    $255, %eax
>       .p2align 4,,10
> @@ -282,7 +282,7 @@ SYM(GTM_longjmp):
>       incsspd %eax
>       subl    $255, %ecx
>       cmpl    $255, %ecx
> -     jg      .L4
> +     ja      .L4
>       popl    %eax
>  .L3:
>       incsspd %ecx
> --
> 2.14.3

Reply via email to