Hi Segher, Yes, that's why the PR is still open 😊 The ICE can be reproduced with a much simpler testcase which is the one we use for repro in the PR, which is now a P1.
The reason for changing this testcase was that it was invalid code. Regards, Tamar -----Original Message----- From: Segher Boessenkool <seg...@kernel.crashing.org> Sent: Sunday, January 20, 2019 3:48 PM To: Tamar Christina <tamar.christ...@arm.com> Cc: gcc-patches@gcc.gnu.org; nd <n...@arm.com>; Ramana Radhakrishnan <ramana.radhakrish...@arm.com>; Richard Earnshaw <richard.earns...@arm.com>; ni...@redhat.com; Kyrylo Tkachov <kyrylo.tkac...@arm.com> Subject: Re: [PATCH][GCC][Arm] Rewrite arm testcase to use intrinsics Hi! On Thu, Jan 17, 2019 at 03:02:00PM +0000, Tamar Christina wrote: > This test was added back when builtins were being used instead of ACLE > intrinsics. The test as far as I can tell is really testing vcombine, > however some of these builtins no longer exist and causes an ICE. > > This fixes the testcase by changing it to use neon intrinsics. Shouldn't the ICE be fixed as well? [ Sorry if you send a separate patch for that and I missed it ]. Segher
