This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [patch, sim] Add isel instruction
- From: "Abid, Hafiz" <hafiz_abid at mentor dot com>
- To: Mike Frysinger <vapier at gentoo dot org>
- Cc: Joel Brobecker <brobecker at adacore dot com>, <gdb-patches at sourceware dot org>
- Date: Fri, 3 May 2013 08:26:19 +0100
- Subject: Re: [patch, sim] Add isel instruction
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;
-
-#