On Tue, Nov 10, 2015 at 11:50:12AM -0600, Evandro Menezes wrote: > 2015-11-10 Evandro Menezes <e.mene...@samsung.com> > > gcc/ > > * config/aarch64/aarch64.md (predicated): Copy attribute from > "arm.md". > > This patch duplicates an attribute from arm.md so that the same > pipeline model can be used for both AArch32 and AArch64. > > Bootstrapped on arm-unknown-linux-gnueabihf, aarch64-unknown-linux-gnu. > > Please, commit if it's alright. > > -- > Evandro Menezes > >
> From 3b643a3c026350864713e1700dc44e4794d93809 Mon Sep 17 00:00:00 2001 > From: Evandro Menezes <e.mene...@samsung.com> > Date: Mon, 9 Nov 2015 17:11:16 -0600 > Subject: [PATCH 1/2] [AArch64] Add attribute for compatibility with ARM > pipeline models > > gcc/ > * config/aarch64/aarch64.md (predicated): Copy attribute from "arm.md". > --- > gcc/config/aarch64/aarch64.md | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/gcc/config/aarch64/aarch64.md b/gcc/config/aarch64/aarch64.md > index 6b08850..2bc2ff5 100644 > --- a/gcc/config/aarch64/aarch64.md > +++ b/gcc/config/aarch64/aarch64.md > @@ -195,6 +195,11 @@ > ;; 1 :=: yes > (define_attr "far_branch" "" (const_int 0)) > > +;; [For compatibility with ARM in pipeline models] > +;; Attribute that specifies whether or not the instruction is executed > +;; conditionally (<C> != "AL"? "yes": "no"). I'm not sure this <C> != "AL" [...] part makes sense to me (thinking only of AArch64, I'd understand it on AArch32 :) ) and we should document that this is never set for AArch64. Could you respin with a slightly clearer comment. Thanks, James