On Mon, 21 Oct 2013, Tobias Burnus wrote: > Dear all, > > attached is a new version of the patch. Changes: > * "#pragma GCC ivdep" instead of "#pragma ivdep" > * Corrections to the error message in c-parser.c and a test case for it > * New wording in the .texi and examples > > I am still not completely happy with the wording ? and I am open for > suggestions. In the example, I played safe and mention k < -m and k >=m; even > if k >= 0 probably works. > > I also didn't know how to best state the reason for requiring a condition. > (Internal reason: The annotation is attached to the condition - thus, it has > to be present. External reason: For vectorization, there shouldn't be a > branching in the body of the loop and without a condition in either the "for" > header or in its body, one has an endless loop.) > > Do you have suggestions for a better wording? If not, is the patch okay for > the trunk? > Built and regtested (C only). [An all-language bootstrap + regtesting is > underway.]
The C front-end changes in this version are OK. -- Joseph S. Myers jos...@codesourcery.com