public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* RE: PING: [PATCH, ARM, iWMMXt][5/5]: pipeline description
@ 2011-07-29  5:13 Xinyu Qi
  0 siblings, 0 replies; 5+ messages in thread
From: Xinyu Qi @ 2011-07-29  5:13 UTC (permalink / raw)
  To: gcc-patches

Ping.

http://gcc.gnu.org/ml/gcc-patches/2011-07/msg01106.html

At 2011-07-14 15:50:55,"Xinyu Qi" <xyqi@marvell.com> wrote:
> 
> > Hi,
> >
> > It is the fifth part of iWMMXt maintenance.
> >
> 
> *config/arm/t-arm (MD_INCLUDES): Add marvell-f-iwmmxt.md.
> *config/arm/marvell-f-iwmmxt.md: New file.
> *config/arm/arm.md (marvell-f-iwmmxt.md): Include.
> 
> 
> Thanks,
> Xinyu

^ permalink raw reply	[flat|nested] 5+ messages in thread

* RE: PING: [PATCH, ARM, iWMMXt][5/5]: pipeline description
  2011-10-20 13:39 ` Ramana Radhakrishnan
@ 2011-12-22  7:56   ` Xinyu Qi
  0 siblings, 0 replies; 5+ messages in thread
From: Xinyu Qi @ 2011-12-22  7:56 UTC (permalink / raw)
  To: Ramana Radhakrishnan; +Cc: gcc-patches, Richard Earnshaw

[-- Attachment #1: Type: text/plain, Size: 890 bytes --]

At 2011-10-20 20:36:53,"Ramana Radhakrishnan" <ramana.radhakrishnan@linaro.org> wrote:  
> On 20 October 2011 08:42, Xinyu Qi <xyqi@marvell.com> wrote:
> > Ping
> >
> > http://gcc.gnu.org/ml/gcc-patches/2011-07/msg01106.html
> > Index: gcc/config/arm/marvell-f-iwmmxt.md
> >
> ================================================================
> ===
> > --- gcc/config/arm/marvell-f-iwmmxt.md	(revision 0)
> > +++ gcc/config/arm/marvell-f-iwmmxt.md	(revision 0)
> >@@ -0,0 +1,179 @@
> >+;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
> >+;; instructions classes
> 
> s/instructions/Instruction.
> 
> Otherwise OK.
> 
> Ramana

Fix the typo. New diff is attached.

	* config/arm/t-arm (MD_INCLUDES): Add marvell-f-iwmmxt.md.
	* config/arm/marvell-f-iwmmxt.md: New file.
	* config/arm/arm.md (marvell-f-iwmmxt.md): Include.

Thanks,
Xinyu

[-- Attachment #2: 5_pipeline.diff --]
[-- Type: application/octet-stream, Size: 7491 bytes --]

Index: gcc/config/arm/t-arm
===================================================================
--- gcc/config/arm/t-arm	(revision 182518)
+++ gcc/config/arm/t-arm	(working copy)
@@ -50,6 +50,7 @@ MD_INCLUDES=	$(srcdir)/config/arm/arm102
 		$(srcdir)/config/arm/iterators.md \
 		$(srcdir)/config/arm/iwmmxt.md \
 		$(srcdir)/config/arm/ldmstm.md \
+		$(srcdir)/config/arm/marvell-f-iwmmxt.md \
 		$(srcdir)/config/arm/neon.md \
 		$(srcdir)/config/arm/predicates.md \
 		$(srcdir)/config/arm/sync.md \
Index: gcc/config/arm/marvell-f-iwmmxt.md
===================================================================
--- gcc/config/arm/marvell-f-iwmmxt.md	(revision 0)
+++ gcc/config/arm/marvell-f-iwmmxt.md	(revision 0)
@@ -0,0 +1,179 @@
+;; Marvell WMMX2 pipeline description
+;; Copyright (C) 2011 Free Software Foundation, Inc.
+;; Written by Marvell, Inc.
+
+;; This file is part of GCC.
+
+;; GCC is free software; you can redistribute it and/or modify it
+;; under the terms of the GNU General Public License as published
+;; by the Free Software Foundation; either version 3, or (at your
+;; option) any later version.
+
+;; GCC is distributed in the hope that it will be useful, but WITHOUT
+;; ANY WARRANTY; without even the implied warranty of MERCHANTABILITY
+;; or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public
+;; License for more details.
+
+;; You should have received a copy of the GNU General Public License
+;; along with GCC; see the file COPYING3.  If not see
+;; <http://www.gnu.org/licenses/>.
+
+
+(define_automaton "marvell_f_iwmmxt")
+
+;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
+;; Pipelines
+;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
+
+;; This is a 7-stage pipelines:
+;;
+;;    MD | MI | ME1 | ME2 | ME3 | ME4 | MW
+;;
+;; There are various bypasses modelled to a greater or lesser extent.
+;;
+;; Latencies in this file correspond to the number of cycles after
+;; the issue stage that it takes for the result of the instruction to
+;; be computed, or for its side-effects to occur.
+
+(define_cpu_unit "mf_iwmmxt_MD" "marvell_f_iwmmxt")
+(define_cpu_unit "mf_iwmmxt_MI" "marvell_f_iwmmxt")
+(define_cpu_unit "mf_iwmmxt_ME1" "marvell_f_iwmmxt")
+(define_cpu_unit "mf_iwmmxt_ME2" "marvell_f_iwmmxt")
+(define_cpu_unit "mf_iwmmxt_ME3" "marvell_f_iwmmxt")
+(define_cpu_unit "mf_iwmmxt_ME4" "marvell_f_iwmmxt")
+(define_cpu_unit "mf_iwmmxt_MW" "marvell_f_iwmmxt")
+
+(define_reservation "mf_iwmmxt_ME"
+      "mf_iwmmxt_ME1,mf_iwmmxt_ME2,mf_iwmmxt_ME3,mf_iwmmxt_ME4"
+)
+
+(define_reservation "mf_iwmmxt_pipeline"
+      "mf_iwmmxt_MD, mf_iwmmxt_MI, mf_iwmmxt_ME, mf_iwmmxt_MW"
+)
+
+;; An attribute to indicate whether our reservations are applicable.
+(define_attr "marvell_f_iwmmxt" "yes,no"
+  (const (if_then_else (symbol_ref "arm_arch_iwmmxt")
+                       (const_string "yes") (const_string "no"))))
+
+;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
+;; instruction classes
+;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
+
+;; An attribute appended to instructions for classification
+
+(define_attr "wmmxt_shift" "yes,no"
+  (if_then_else (eq_attr "wtype" "wror, wsll, wsra, wsrl")
+		(const_string "yes") (const_string "no"))
+)
+
+(define_attr "wmmxt_pack" "yes,no"
+  (if_then_else (eq_attr "wtype" "waligni, walignr, wmerge, wpack, wshufh, wunpckeh, wunpckih, wunpckel, wunpckil")
+		(const_string "yes") (const_string "no"))
+)
+
+(define_attr "wmmxt_mult_c1" "yes,no"
+  (if_then_else (eq_attr "wtype" "wmac, wmadd, wmiaxy, wmiawxy, wmulw, wqmiaxy, wqmulwm")
+		(const_string "yes") (const_string "no"))
+)
+
+(define_attr "wmmxt_mult_c2" "yes,no"
+  (if_then_else (eq_attr "wtype" "wmul, wqmulm")
+		(const_string "yes") (const_string "no"))
+)
+
+(define_attr "wmmxt_alu_c1" "yes,no"
+  (if_then_else (eq_attr "wtype" "wabs, wabsdiff, wand, wandn, wmov, wor, wxor")
+	        (const_string "yes") (const_string "no"))
+)
+
+(define_attr "wmmxt_alu_c2" "yes,no"
+  (if_then_else (eq_attr "wtype" "wacc, wadd, waddsubhx, wavg2, wavg4, wcmpeq, wcmpgt, wmax, wmin, wsub, waddbhus, wsubaddhx")
+		(const_string "yes") (const_string "no"))
+)
+
+(define_attr "wmmxt_alu_c3" "yes,no"
+  (if_then_else (eq_attr "wtype" "wsad")
+	        (const_string "yes") (const_string "no"))
+)
+
+(define_attr "wmmxt_transfer_c1" "yes,no"
+  (if_then_else (eq_attr "wtype" "tbcst, tinsr, tmcr, tmcrr")
+                (const_string "yes") (const_string "no"))
+)
+
+(define_attr "wmmxt_transfer_c2" "yes,no"
+  (if_then_else (eq_attr "wtype" "textrm, tmovmsk, tmrc, tmrrc")
+	        (const_string "yes") (const_string "no"))
+)
+
+(define_attr "wmmxt_transfer_c3" "yes,no"
+  (if_then_else (eq_attr "wtype" "tmia, tmiaph, tmiaxy")
+	        (const_string "yes") (const_string "no"))
+)
+
+;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
+;; Main description
+;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
+
+(define_insn_reservation "marvell_f_iwmmxt_alu_c1" 1
+  (and (eq_attr "marvell_f_iwmmxt" "yes")
+       (eq_attr "wmmxt_alu_c1" "yes"))
+  "mf_iwmmxt_pipeline")
+
+(define_insn_reservation "marvell_f_iwmmxt_pack" 1
+  (and (eq_attr "marvell_f_iwmmxt" "yes")
+       (eq_attr "wmmxt_pack" "yes"))
+  "mf_iwmmxt_pipeline")
+
+(define_insn_reservation "marvell_f_iwmmxt_shift" 1
+  (and (eq_attr "marvell_f_iwmmxt" "yes")
+       (eq_attr "wmmxt_shift" "yes"))
+  "mf_iwmmxt_pipeline")
+
+(define_insn_reservation "marvell_f_iwmmxt_transfer_c1" 1
+  (and (eq_attr "marvell_f_iwmmxt" "yes")
+       (eq_attr "wmmxt_transfer_c1" "yes"))
+  "mf_iwmmxt_pipeline")
+
+(define_insn_reservation "marvell_f_iwmmxt_transfer_c2" 5
+  (and (eq_attr "marvell_f_iwmmxt" "yes")
+       (eq_attr "wmmxt_transfer_c2" "yes"))
+  "mf_iwmmxt_pipeline")
+
+(define_insn_reservation "marvell_f_iwmmxt_alu_c2" 2
+  (and (eq_attr "marvell_f_iwmmxt" "yes")
+       (eq_attr "wmmxt_alu_c2" "yes"))
+  "mf_iwmmxt_pipeline")
+
+(define_insn_reservation "marvell_f_iwmmxt_alu_c3" 3
+  (and (eq_attr "marvell_f_iwmmxt" "yes")
+       (eq_attr "wmmxt_alu_c3" "yes"))
+  "mf_iwmmxt_pipeline")
+
+(define_insn_reservation "marvell_f_iwmmxt_transfer_c3" 4
+  (and (eq_attr "marvell_f_iwmmxt" "yes")
+       (eq_attr "wmmxt_transfer_c3" "yes"))
+  "mf_iwmmxt_pipeline")
+
+(define_insn_reservation "marvell_f_iwmmxt_mult_c1" 4
+  (and (eq_attr "marvell_f_iwmmxt" "yes")
+       (eq_attr "wmmxt_mult_c1" "yes"))
+  "mf_iwmmxt_pipeline")
+
+;There is a forwarding path from ME3 stage
+(define_insn_reservation "marvell_f_iwmmxt_mult_c2" 3
+  (and (eq_attr "marvell_f_iwmmxt" "yes")
+       (eq_attr "wmmxt_mult_c2" "yes"))
+  "mf_iwmmxt_pipeline")
+
+(define_insn_reservation "marvell_f_iwmmxt_wstr" 0
+  (and (eq_attr "marvell_f_iwmmxt" "yes")
+       (eq_attr "wtype" "wstr"))
+  "mf_iwmmxt_pipeline")
+
+;There is a forwarding path from MW stage
+(define_insn_reservation "marvell_f_iwmmxt_wldr" 5
+  (and (eq_attr "marvell_f_iwmmxt" "yes")
+       (eq_attr "wtype" "wldr"))
+  "mf_iwmmxt_pipeline")
Index: gcc/config/arm/arm.md
===================================================================
--- gcc/config/arm/arm.md	(revision 182518)
+++ gcc/config/arm/arm.md	(working copy)
@@ -538,6 +538,7 @@ (define_attr "generic_vfp" "yes,no"
 	  (const_string "yes")
 	  (const_string "no"))))
 
+(include "marvell-f-iwmmxt.md")
 (include "arm-generic.md")
 (include "arm926ejs.md")
 (include "arm1020e.md")

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: PING: [PATCH, ARM, iWMMXt][5/5]: pipeline description
  2011-10-20  8:15 Xinyu Qi
@ 2011-10-20 13:39 ` Ramana Radhakrishnan
  2011-12-22  7:56   ` Xinyu Qi
  0 siblings, 1 reply; 5+ messages in thread
From: Ramana Radhakrishnan @ 2011-10-20 13:39 UTC (permalink / raw)
  To: Xinyu Qi; +Cc: gcc-patches

On 20 October 2011 08:42, Xinyu Qi <xyqi@marvell.com> wrote:
> Ping
>
> http://gcc.gnu.org/ml/gcc-patches/2011-07/msg01106.html
> Index: gcc/config/arm/marvell-f-iwmmxt.md
> ===================================================================
> --- gcc/config/arm/marvell-f-iwmmxt.md	(revision 0)
> +++ gcc/config/arm/marvell-f-iwmmxt.md	(revision 0)
>@@ -0,0 +1,179 @@
>+;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
>+;; instructions classes

s/instructions/Instruction.

Otherwise OK.

Ramana

^ permalink raw reply	[flat|nested] 5+ messages in thread

* RE: PING: [PATCH, ARM, iWMMXt][5/5]: pipeline description
@ 2011-10-20  8:15 Xinyu Qi
  2011-10-20 13:39 ` Ramana Radhakrishnan
  0 siblings, 1 reply; 5+ messages in thread
From: Xinyu Qi @ 2011-10-20  8:15 UTC (permalink / raw)
  To: Ramana Radhakrishnan, gcc-patches

Ping

http://gcc.gnu.org/ml/gcc-patches/2011-07/msg01106.html

	* config/arm/t-arm (MD_INCLUDES): Add marvell-f-iwmmxt.md.
	* config/arm/marvell-f-iwmmxt.md: New file.
	* config/arm/arm.md (marvell-f-iwmmxt.md): Include.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* RE: PING: [PATCH, ARM, iWMMXt][5/5]: pipeline description
@ 2011-09-26  7:08 Xinyu Qi
  0 siblings, 0 replies; 5+ messages in thread
From: Xinyu Qi @ 2011-09-26  7:08 UTC (permalink / raw)
  To: Ramana Radhakrishnan; +Cc: gcc-patches

Ping.

http://gcc.gnu.org/ml/gcc-patches/2011-07/msg01106.html

	* config/arm/t-arm (MD_INCLUDES): Add marvell-f-iwmmxt.md.
	* config/arm/marvell-f-iwmmxt.md: New file.
	* config/arm/arm.md (marvell-f-iwmmxt.md): Include.

At 2011-07-29 11:09:37,"Xinyu Qi" <xyqi@marvell.com> wrote: > Ping.
> 
> http://gcc.gnu.org/ml/gcc-patches/2011-07/msg01106.html
> 
> At 2011-07-14 15:50:55,"Xinyu Qi" <xyqi@marvell.com> wrote:
> >
> > > Hi,
> > >
> > > It is the fifth part of iWMMXt maintenance.
> > >
> >
> > *config/arm/t-arm (MD_INCLUDES): Add marvell-f-iwmmxt.md.
> > *config/arm/marvell-f-iwmmxt.md: New file.
> > *config/arm/arm.md (marvell-f-iwmmxt.md): Include.
> >
> >
> > Thanks,
> > Xinyu

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2011-12-22  6:55 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-07-29  5:13 PING: [PATCH, ARM, iWMMXt][5/5]: pipeline description Xinyu Qi
2011-09-26  7:08 Xinyu Qi
2011-10-20  8:15 Xinyu Qi
2011-10-20 13:39 ` Ramana Radhakrishnan
2011-12-22  7:56   ` Xinyu Qi

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).