On 8 February 2015 at 02:24, Andrew Pinski <pins...@gmail.com> wrote:
> Here is the updated patch with Jakub's comments included and added a > testcase for the 0, 0 case. > > Thanks, > Andrew Pinski > ChangeLog: > > PR target/64893 > * config/aarch64/aarch64-builtins.c (aarch64_init_simd_builtins): > Change the first argument type to size_type_node and add another > size_type_node. > (aarch64_simd_expand_builtin): Handle the new argument to > AARCH64_SIMD_BUILTIN_LANE_CHECK and don't ICE but rather > print an out when the first two arguments are not > nonzero integer constants. > * config/aarch64/arm_neon.h (__AARCH64_LANE_CHECK): > Pass the sizeof directly to __builtin_aarch64_im_lane_boundsi. > > testsuite/ChangeLog: > > * c-c++-common/torture/aarch64-vect-lane-1.c: New testcase. > * c-c++-common/torture/aarch64-vect-lane-2.c: New testcase. OK, thanks /Marcus
