On Tue, 19 Mar 2019 08:47:49 +0000 Richard Sandiford <richard.sandif...@arm.com> wrote:
> > ... this identical condition, AFAICS? > > So this second conditions else arm should be dead, shouldn't it? > > Yeah, that's what: > > /* ??? The "if" arm is written to handle all cases; see below for what > we would do for !LOOP_VINFO_FULLY_MASKED_P. */ Ok, i somehow managed not to correlate this comment to the inner if's else. It's been late. Sorry for the noise! thanks, > > was trying to say. Like I mentioned in the covering note, in principle > the approach of calculating the minimum number of vector iterations > should work for all cases, and we might want to consider doing that > for stage 1. I wanted to show what the !LOOP_VINFO_FULLY_MASKED_P > code would look like if we did that. > > I'd wondered about putting the inner else in an #if 0 or comment instead, > but this way makes it easier to experiment with. > > Thanks, > Richard