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.


Otherwise LGTM.

I think all your other comments are spot on and will address. Do you want another review with them fixed?

If not, I think the only thing remaining is the IFN_UNIQUE patch, which (At least) needs an update to use targetm.have... conversion.

nathan

Reply via email to