Ping.

Fixes for the review comments by Aaron.

Thanks, I've commited the changes in revision 248158 (but forgot to add link in svn commit message).
Alexander

Added an assertion for TiedTo value.

Hi Aaron,

John, thank you, I've fixed all the issues.

Hi John,

Hi John,

Ping.

Closed by commit rL218743 (authored by @amusman).

I updated the code to make the number of iterations 64-bit if we cannot prove that 32 bits would be enough and added a test with such example.
I'm not sure if it's worth to do widening to 128 bits in similar cases with several nested loops with 64-bit counters - in particular, in case of worksharing loop directives the library only supports 32-bit and 64-bit counters, so we'll need truncate it to 64 bits for such directives anyway.

Closed by commit rL218660 (authored by @amusman).

Here is updated patch, I added type parameter to getOpenMPSimdDefaultAlignment (I guess you meant llvm::Type, not clang::Type) and fixed other issues.

Hi Hal,

Closed by commit rL218503 (authored by @amusman).

Ping.

Ping.

Hello Sunita,

Closed by commit rL210095 (authored by @amusman).

Here is updated patch.

Hi Richard,

Yes, I think that assuming that the variable is random access iterator would work. It will still need to generate std::difference on them, but we can check that later, during building expressions for iteration space (that will be used for CodeGen). This also has an advantage that user may use his iterators without deriving from std classes.
I've removed corresponding checks and fixed processing break statements.

Thank you for review.

Closed by commit rL209411 (authored by @amusman).

Committed, revision 206956.