Re: [patch, sim] Add isel instruction

Abid, Hafiz
2013-05-03T07:26:25Z

On 30/04/13 18:09:41, Mike Frysinger wrote:
> On Tuesday 30 April 2013 07:12:35 Abid, Hafiz wrote:
> > On 30/04/13 12:05:14, Joel Brobecker wrote:
> > > > Thansk for lettting me know. If this is urgent for you then I
> can
> > > > revert as it may take me some time to find a fix.
> > >
> > > Do you have an idea of what "some time" would mean?
> >
> > By the end of this week.
>
> i'm fine with that ... if it goes longer, then let's punt it
> -mike

It looks that isel was already supported in the simulator for the e500.
The duplication caused the error that Joel observed. I will revert now
with the following.

Thanks,
Abid

sim/ppc/ChangeLog:
2013-05-03 Hafiz Abid Qadeer <abidh@codesourcery.com>

revert:
2013-04-19 Nathan Froyd <froydnj@codesourcery.com>
* ppc-instructions (isel): New instruction.

diff --git a/sim/ppc/ppc-instructions b/sim/ppc/ppc-instructions
index d76a765..1b8fd89 100644
--- a/sim/ppc/ppc-instructions
+++ b/sim/ppc/ppc-instructions
@@ -3455,19 +3455,6 @@
void::function::invalid_zero_divide_operation:cpu *proces
PPC_INSN_MFCR(RT_BITMASK);

#
-# I.3.3.15 Fixed-Point Select
-#
-
-0.31,6.RT,11.RA,16.RB,21.BF,26.15,31./:A::isel:Integer Select
- unsigned_word a;
- if (RA_is_0) a = 0;
- else a = *rA;
- if (CR & (1 << (31 - BF)))
- *rT = a;
- else
- *rT = *rB;
-
-#