On Fri, 22 Mar 2019 04:39:29 +0900,
Richard Henderson wrote:
>
> On 3/20/19 7:16 AM, Yoshinori Sato wrote:
> > +static const char *cond[] = {
> > + "eq", "ne", "c", "nc", "gtu", "leu", "pz", "n",
> > + "ge", "lt", "gt", "le", "o", "no", "ra", "f"
> > +};
>
> const char * const cond[]
>
> Or since they are all very short strings,
>
> const char cond[][4]
OK.
> > +static const char *cr[] = {
> > + "psw", "", "usp", "fpsw", "", "", "", "",
> > + "bpsw", "bpc", "isp", "fintv", "intb", "", "", "",
> > +};
> > +static const char *msize[] = {
> > + "b", "w", "l", "ub", "uw",
> > +};
>
> Similarly. Although cr has longer strings. And cr[1] = "pc".
OK.
I forgot it.
> > +static void operand(DisasContext *ctx, int ld, int mi, int rs, int rd)
> > +{
> > + int dsp;
> > + const char *mis;
> > + static const char *sizes[] = {".b", ".w", ".l"};
>
> Similarly. Although why not include .ub and .uw here?
>
It set mi == 4 or mi == 5 case.
But it confused. Fixed.
>
> r~
>
--
Yosinori Sato
