Re: m68k_register_virtual_type

[Andreas Schwab <schwab@suse.de>]

Andrew Cagney <ac131313@redhat.com> writes:

|> > |> > +  else if ((unsigned) regnum >= E_FPC_REGNUM)
|> > |> > +    return builtin_type_int;
|> > |> |> builtin_type_int32 is better
|> > |> |> >    else if ((unsigned) regnum >= FP0_REGNUM)
|> > |> >      return builtin_type_long_double;
|> > |> > +  else if (regnum == PS_REGNUM)
|> > |> > +    return builtin_type_int
|> > |> |> Ditto.
|> > What about the default case for the d0-d7 registers?  Should it be
|> > changed
|> > to builtin_type_int32 as well?
|> 
|> Er, yes.  (And eventually long_double to builtin_type_m68881_ext but the
|> last one might have fallout related to register conversions).

What kind of fallout could that be?  Some quick tests didn't reveal
anything obvious.

Andreas.

-- 
Andreas Schwab, SuSE Labs, schwab@suse.de
SuSE Linux AG, Deutschherrnstr. 15-19, D-90429 Nürnberg
Key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."