This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Re: m68k_register_virtual_type
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."