On Thu, Mar 20, 2014 at 12:59:27PM +0000, Marcus Shawcroft wrote: > On 25 February 2014 11:58, Alex Velenko <alex.vele...@arm.com> wrote: > > Hi, > > > > This patch fixes a bug in vshr_n_u64 and vshrd_n_u64 intrinsic > > behavior in case of shift by 64. Shift by 64 is strictly defined in ACLE to > > use ushr instruction intended by those intrinsics. > > > > The testcase provided also tests the behavior for intrinsics mentioned > > above with values other then 64. Besides, the test checks that an illeagal > > ushr shift by 0 is not generated, expecting the test to compile and run > > correctly generating instructions other than ushr. > > > > The patch was tested for LE and BE with no regressions. > > > > Is given patch ok for stage-4? > > I think this is OK for stage-4, but leave 24h before committing to > allow the RM's opportunity to object / comment. >
I've committed this on Alex' behalf as revision 208789. Thanks, James
