On 20 November 2014 07:49, Marcus Shawcroft <marcus.shawcr...@gmail.com> wrote: > On 19 November 2014 19:05, Charles Baylis <charles.bay...@linaro.org> wrote: > >> PR target/63870 >> * config/aarch64/aarch64-builtins.c (aarch64_simd_expand_args): Pass >> expression to aarch64_simd_lane_bounds. >> * config/aarch64/aarch64-protos.h (aarch64_simd_lane_bounds): Update >> prototype. >> * config/aarch64/aarch64.c (aarch64_simd_lane_bounds): Add exp >> parameter. Report calling function in error message if exp is >> non-NULL. > > These needs to be updated to reflect the changes in the last revision > of the patch where NULL is passed explicitly. Otherwise OK, commit it > with a fixed ChangeLog.
Sorry... more haste, less speed. Committed as r217885, with the following ChangeLog: 2014-11-20 Charles Baylis <charles.bay...@linaro.org> PR target/63870 * config/aarch64/aarch64-builtins.c (aarch64_simd_expand_args): Pass expression to aarch64_simd_lane_bounds. * config/aarch64/aarch64-protos.h (aarch64_simd_lane_bounds): Update prototype. * config/aarch64/aarch64-simd.md: (aarch64_combinez<mode>): Update call to aarch64_simd_lane_bounds. (aarch64_get_lanedi): Likewise. (aarch64_ld2_lane<mode>): Likewise. (aarch64_ld3_lane<mode>): Likewise. (aarch64_ld4_lane<mode>): Likewise. (aarch64_im_lane_boundsi): Likewise. * config/aarch64/aarch64.c (aarch64_simd_lane_bounds): Add exp parameter. Report calling function in error message if exp is non-NULL.