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
> 
> 2016-06-06  Kyrylo Tkachov  <kyrylo.tkac...@arm.com>
> 
>     * config/arm/cortex-a57.md (cortex_a57_alu):
>     Handle csel type.


This is ok by me - in these sorts of situations I think we should let the 
aarch64 approvals be sufficient.

regards
Ramana

Reply via email to