Re: [PATCH 2/2, AArch64, v2] Pipeline model for APM XGene-1.

2014-11-20 Thread Ramana Radhakrishnan
On Wed, Nov 19, 2014 at 9:42 PM, Philipp Tomsich wrote: > Here's an updated patch with Kyrill's and Andrew's comments integrated. > > I left the file in the config/arm-directory, as XGene-family is capable of > executing ARMv7 and we will wire this into the 32bit backend in the near > future (movi

Re: [PATCH 2/2, AArch64, v2] Pipeline model for APM XGene-1.

2014-11-20 Thread Kyrill Tkachov
Hi Philipp, On 20/11/14 10:47, Dr. Philipp Tomsich wrote: Kyrill, I don't mind it being in config/arm if you plan to wire it up later, good to know. Another comment inline…. I’ll clean up the missing xgene1_ and the mistyped xgene_ prefix and resubmit. +(define_insn_reservation "div" 2 +

Re: [PATCH 2/2, AArch64, v2] Pipeline model for APM XGene-1.

2014-11-20 Thread Dr . Philipp Tomsich
Kyrill, > I don't mind it being in config/arm if you plan to wire it up later, good to > know. > Another comment inline…. I’ll clean up the missing xgene1_ and the mistyped xgene_ prefix and resubmit. >> +(define_insn_reservation "div" 2 >> + (and (eq_attr "tune" "xgene1") >> + (eq_attr

Re: [PATCH 2/2, AArch64, v2] Pipeline model for APM XGene-1.

2014-11-20 Thread Kyrill Tkachov
Hi Philipp, I don't mind it being in config/arm if you plan to wire it up later, good to know. Another comment inline Thanks, Kyrill On 19/11/14 21:42, Philipp Tomsich wrote: Here's an updated patch with Kyrill's and Andrew's comments integrated. I left the file in the config/arm-direct

[PATCH 2/2, AArch64, v2] Pipeline model for APM XGene-1.

2014-11-19 Thread Philipp Tomsich
Here's an updated patch with Kyrill's and Andrew's comments integrated. I left the file in the config/arm-directory, as XGene-family is capable of executing ARMv7 and we will wire this into the 32bit backend in the near future (moving it now would just cause another move in the near future). We