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
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
+
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
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
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