On Tue, 19 Nov 2024 at 15:00, Andre Vieira (lists) <andre.simoesdiasvie...@arm.com> wrote: > > Hi, > > Looks like single_pred ICEs if the basic-block does not have a single > predecessor rather than return NULL, which was what this snippet of code > relied on. > This feels like borderline obvious to me as a fix, but I thought I'd get > it checked by one more person. > > Call 'single_pred_p' before 'single_pred' to verify it is safe to do so. >
The patch is OK, thanks. Christophe > gcc/ChangeLog: > > * config/arm/arm.cc (arm_mve_dlstp_check_dec_counter): Call > single_pred_p to verify it's safe to call single_pred. > > gcc/testsuite/ChangeLog: > > * gcc.target/arm/mve/dlstp-loop-form.c: Add loop that triggered ICE.
