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

2014-11-19 Thread Dr. Philipp Tomsich
Kyrill, The XGene-cores all support AArch32. We plan to wire this up, but I’d like to merge this as-is (i.e. without wiring it up for AArch32) initially, as we haven’t done the same amount of QA on these with AArch32 as with AArch64. Do you have any issue with this going into the config/arm-dir

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

2014-11-19 Thread Andrew Pinski
A few other comments. On Wed, Nov 19, 2014 at 9:32 AM, Philipp Tomsich wrote: > --- > gcc/ChangeLog | 6 + > gcc/config/aarch64/aarch64.md | 4 +- > gcc/config/arm/xgene1.md | 739 > ++ > 3 files changed, 748 insertions(+), 1 dele

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

2014-11-19 Thread Kyrill Tkachov
Hi Philipp, One more comment... On 19/11/14 17:32, Philipp Tomsich wrote: @@ -4211,3 +4211,5 @@ ;; Atomic Operations (include "atomics.md") + +(include "../arm/xgene1.md") Do you expect to add arm support for this core? If so, you should wire it up there as well. If not, I think Andrew

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

2014-11-19 Thread Kyrill Tkachov
Hi Philipp, Some comments inline On 19/11/14 17:32, Philipp Tomsich wrote: --- gcc/ChangeLog | 6 + gcc/config/aarch64/aarch64.md | 4 +- gcc/config/arm/xgene1.md | 739 ++ 3 files changed, 748 insertions(+), 1 deletion(-)

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

2014-11-19 Thread Philipp Tomsich
--- gcc/ChangeLog | 6 + gcc/config/aarch64/aarch64.md | 4 +- gcc/config/arm/xgene1.md | 739 ++ 3 files changed, 748 insertions(+), 1 deletion(-) create mode 100644 gcc/config/arm/xgene1.md diff --git a/gcc/ChangeLog b/gcc/Change