This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch 1/2][ARM]: New CPU support for Marvell Whitney
- From: Kyrill Tkachov <kyrylo dot tkachov at arm dot com>
- To: Xingxing Pan <xxingpan at marvell dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Cc: Ramana Radhakrishnan <Ramana dot Radhakrishnan at arm dot com>, Richard Earnshaw <Richard dot Earnshaw at arm dot com>, "xyqi at marvell dot com" <xyqi at marvell dot com>, "lgao1 at marvell dot com" <lgao1 at marvell dot com>
- Date: Fri, 19 Dec 2014 10:36:39 +0000
- Subject: Re: [patch 1/2][ARM]: New CPU support for Marvell Whitney
- Authentication-results: sourceware.org; auth=none
- References: <5492A8AD dot 1030203 at marvell dot com>
Hi Xingxing,
On 18/12/14 10:13, Xingxing Pan wrote:
+
+(define_attr "tune_marvell_whitney" "yes,no"
+ (const (if_then_else (eq_attr "tune" "marvell_whitney")
+ (const_string "yes")
+ (const_string "no"))))
Why do you need this? The canonical way we do this in the other .md
files is check
for the "tune" attribute directly.
+
+(define_automaton "whitney_pipe")
+
+(define_cpu_unit "wCU1" "whitney_pipe")
+(define_cpu_unit "wCU2" "whitney_pipe")
+(define_cpu_unit "wCU3" "whitney_pipe")
+(define_cpu_unit "wCU4" "whitney_pipe")
+(define_cpu_unit "wCU5" "whitney_pipe")
+(define_cpu_unit "wCU6" "whitney_pipe")
+(define_cpu_unit "wCU7" "whitney_pipe")
+(define_cpu_unit "wCU8" "whitney_pipe")
+(define_cpu_unit "wCU9" "whitney_pipe")
+(define_cpu_unit "wCU10" "whitney_pipe")
+(define_cpu_unit "wCU11" "whitney_pipe")
+(define_cpu_unit "wCU12" "whitney_pipe")
+(define_cpu_unit "wCU13" "whitney_pipe")
+
+(define_insn_reservation "wIR1" 1
+ (and (eq_attr "tune_marvell_whitney" "yes")
+ (eq_attr "whitney_type" "wTP1"))
+ "wCU1|wCU2|(wCU3+wCU4)"
+)
Further to the comment above, this could be written as:
+(define_insn_reservation "wIR1" 1
+ (and (eq_attr "tune" "marvell_whitney")
+ (eq_attr "whitney_type" "wTP1"))
+ "wCU1|wCU2|(wCU3+wCU4)"
+)
Same with the other reservations in this file.
Kyrill
+
+(define_insn_reservation "wIR2" 2
+ (and (eq_attr "tune_marvell_whitney" "yes")
+ (eq_attr "whitney_type" "wTP2"))
+ "wCU1|wCU2|(wCU3+wCU4)"
+)