On Mon, Oct 26, 2015 at 09:13:28AM -0700, Nathan Sidwell wrote:
> On 10/26/15 08:13, Jakub Jelinek wrote:
>
> >>It won't convert them into such representations.
> >
> >Can you fix that incrementally? I'd expect that code marked with acc loop
> >vector
> >can't have loop carried backward lexical dependencies, at least not within
> >the adjacent number of iterations specified in vector clause?
>
> Sure. I was using 'won't' to describe the patch, not claiming it could
> never be changed to do that kind of thing.
Ok.
> >Otherwise LGTM.
>
> I think all your other comments are spot on and will address. Do you want
> another review with them fixed?
Just committing fixed version (and posting what you've committed for patches
that changed since the patch that has been posted earlier) is enough.
> If not, I think the only thing remaining is the IFN_UNIQUE patch, which
> (At least) needs an update to use targetm.have... conversion.
Ok, will wait till you make those changes then?
Jakub
