On Fri, Jun 23, 2017 at 10:27:55AM +0100, Michael Collison wrote:
> Fixed the "nitpick" issues pointed out by James. Okay for trunk?


> > I have a few comments below, which are closer to nitpicking than structural
> > issues with the patch.
> > 
> > With those fixed, this is OK to commit.

This is still OK for trunk.

Thanks,
James

> 2017-05-22  Kyrylo Tkachov  <kyrylo.tkac...@arm.com>
>           Michael Collison <michael.colli...@arm.com>
> 
>       PR target/70119
>       * config/aarch64/aarch64.md (*aarch64_<optab>_reg_<mode>3_mask1):
>       New pattern.
>       (*aarch64_reg_<mode>3_neg_mask2): New pattern.
>       (*aarch64_reg_<mode>3_minus_mask): New pattern.
>       (*aarch64_<optab>_reg_di3_mask2): New pattern.
>       * config/aarch64/aarch64.c (aarch64_rtx_costs): Account for cost
>       of shift when the shift amount is masked with constant equal to
>       the size of the mode.
>       * config/aarch64/predicates.md (subreg_lowpart_operator): New
>       predicate.
> 
> 
> 2016-05-22  Kyrylo Tkachov  <kyrylo.tkac...@arm.com>
>           Michael Collison <michael.colli...@arm.com>
> 
>       PR target/70119
>       * gcc.target/aarch64/var_shift_mask_1.c: New test.
> 

Reply via email to