On 6 November 2014 10:19, Alan Lawrence <alan.lawre...@arm.com> wrote: > This generates out-of-range errors at compile- (rather than assemble-)time > for the vqdm*_lane intrinsics, and also provides a single place to do > bigendian lane-swapping for all those intrinsics (and others to follow in > later patches). This allows us to remove many define_expands that just do a > range-check and endian-swap before outputting the RTL for a corresponding > "_internal" insn. > > Changes to aarch64-simd.md are not as big as they look, they are highly > repetitive, like the code they are removing! Testcases are also repetitive, > as unfortunately dg-error doesn't care *how many* errors there were matching > it's pattern, as long as at least 1, hence having to separate each into own > file - the last "0" in the dg-error disables the line-number checking, as > the line numbers in our error messages refer to lines within arm_neon.h > rather than within the test case. (They do at least mention the user > function containing the call to the intrinsic.) > > Ok for trunk?
OK, but make sure the committed patch addresses Mike's comment. /Marcus
