cgen bug with larger-than-normal instructions?
Doug Evans
dje@transmeta.com
Fri Sep 15 08:55:00 GMT 2000
matthew green writes:
> > - first argument of GETIMEM*() is "pc"
> >
> > this allows my decoder to build, but i haven't tested that it works
> > yet (i have now pages of errors for sem.cxx to deal with).
>
> I don't quite follow why adding "pc" is necessary.
>
> well, the function is defined as:
>
> inline UHI GETIMEMUHI(PCADDR pc, IADDR addr) const
>
> so replacing `current_cpu' with `pc' seemed the right thing to do.
Well, my question really was: given that you have `addr',
why is `pc' present at all? The GNU simulators only pass `addr',
so I'm wondering you you also need to pass `pc'.
[for clarity: for the first "element" of the insn, addr == pc,
for subsequent fetches of the same insn addr == pc + n]
More information about the Cgen
mailing list