https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116497

Hongtao Liu <liuhongt at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |liuhongt at gcc dot gnu.org

--- Comment #6 from Hongtao Liu <liuhongt at gcc dot gnu.org> ---
(In reply to Andi Kleen from comment #1)
> Disable check for no_caller_saved_registers enforcing non FP.
> 
> diff --git a/gcc/config/i386/i386-options.cc
> b/gcc/config/i386/i386-options.cc
> index f79257cc764..cec652cc9e6 100644
> --- a/gcc/config/i386/i386-options.cc
> +++ b/gcc/config/i386/i386-options.cc
> @@ -3639,8 +3639,8 @@ ix86_set_current_function (tree fndecl)
>      reinit_regs ();
>  
>    if (cfun->machine->func_type != TYPE_NORMAL
> -      || (cfun->machine->call_saved_registers
> -         == TYPE_NO_CALLER_SAVED_REGISTERS))
> +      /* || (cfun->machine->call_saved_registers
> +        == TYPE_NO_CALLER_SAVED_REGISTERS) */)
>      {
>        /* Don't allow SSE, MMX nor x87 instructions since they
>          may change processor state.  */

I think RA is smart enough to save and restore SSE,MMX or x87 registers, and we
can remove TYPE_NO_CALLER_SAVED_REGISTERS part from this.
Or are there any other concerns here regarding the comments? @hj

Reply via email to