On Tue, Jun 07, 2016 at 09:22:05AM +0100, Ramana Radhakrishnan wrote: > > > On 06/06/16 17:10, Kyrill Tkachov wrote: > > Hi all, > > > > This small patch adds handling of the CSEL-type instructions to the > > Cortex-A57 scheduling model. > > It is treated the same as simple ALU instructions. > > > > With this patch I didn't see any overall differences in SPEC2006. > > > > Bootstrapped and tested on arm-none-linux-gnueabihf and aarch64-linux-gnu. > > > > Ok for trunk? > > > > The patch is very simple and the csel value isn't used in any arm > > instructions so I think > > just an aarch64 approval for this should be enough. > > > > Thanks, > > Kyrill > > > > This is ok by me - in these sorts of situations I think we should let the > aarch64 approvals be sufficient.
I agree. I don't think the folder in which the file resides is the most important thing, it is the intent and risk of the patch that matters. This patch is OK for AArch64. Thanks, James > > 2016-06-06 Kyrylo Tkachov <kyrylo.tkac...@arm.com> > > > > * config/arm/cortex-a57.md (cortex_a57_alu): > > Handle csel type. >
