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