Andrew Carlotti <andrew.carlo...@arm.com> writes:
> @@ -9157,14 +9156,14 @@ aarch64_emit_stack_tie (rtx reg)
>     then the signal handler doesn't know the state of the stack and can make 
> no
>     assumptions about which pages have been probed.
>  
> -   FORCE_ISA_MODE is AARCH64_FL_SM_ON if any variable component of POLY_SIZE
> +   FORCE_ISA_MODE is AARCH64_ISA_MODE_SM_ON if any variable component of 
> POLY_SIZE

Nit: long line (paragraph needs reflowing).

>     is measured relative to the SME vector length instead of the current
>     prevailing vector length.  It is 0 otherwise.  */
>  
>  static void
>  aarch64_allocate_and_probe_stack_space (rtx temp1, rtx temp2,
>                                       poly_int64 poly_size,
> -                                     aarch64_feature_flags force_isa_mode,
> +                                     aarch64_isa_mode force_isa_mode,
>                                       bool frame_related_p,
>                                       bool final_adjustment_p)
>  {
> [...]
> @@ -9877,8 +9876,10 @@ aarch64_expand_epilogue (rtx_call_insn *sibcall)
>      {
>        rtx_insn *guard_label = nullptr;
>        if (known_ge (cfun->machine->frame.old_svcr_offset, 0))
> -     guard_label = aarch64_guard_switch_pstate_sm (IP0_REGNUM,
> -                                                   aarch64_isa_flags);
> +     {
> +       guard_label = aarch64_guard_switch_pstate_sm (IP0_REGNUM,
> +                                                     AARCH64_ISA_MODE);
> +     }

The convention is to avoid braces here.

OK with those changes, thanks.

Richard

Reply via email to