On 07/20/2012 03:34 PM, David Edelsohn wrote:
Looks good. Check it in after testing.
Testing went fine, patch checked in.
-Pat
On Thu, Jul 19, 2012 at 2:43 PM, Pat Haugen wrote:
> Update form stores are cracked into a store and addi, and the store is dual
> issued to the LSU/FXU. So based on this info I would think it should look
> like the following (untested).
>
> @@ -141,12 +141,10 @@ (define_insn_reservation "power4-
On 07/17/2012 12:51 PM, David Edelsohn wrote:
The change to power4-store-update looks incorrect or at least incomplete.
These reservations and others were changed by Vlad in March/April 2004
to fix a consistency check that he introduced at the time. Note that
the dispatch units for the final cho
On Tue, Jul 10, 2012 at 8:20 AM, Steven Bosscher wrote:
> Hello,
>
> These look like typos:
>
> * "power4-store-update" wants "iuX,iuY" for X=1|2 and Y=1|2. The
> "iu2,iu1" case appeared twice.
> * "power4-three" wants "iuX,iuX,iuY|iuX,iuY,iuY" for X=1|2 and Y=1|2.
> The "iu1,iu1,iu2" case appeare
Hello,
These look like typos:
* "power4-store-update" wants "iuX,iuY" for X=1|2 and Y=1|2. The
"iu2,iu1" case appeared twice.
* "power4-three" wants "iuX,iuX,iuY|iuX,iuY,iuY" for X=1|2 and Y=1|2.
The "iu1,iu1,iu2" case appeared twice.
Bootstrapped&tested on powerpc64-unknown-linux-gnu.
OK for tr
