[PATCH] D25764: Add end location of loop to loop metadata.

[Florian Hahn via cfe-commits]

fhahn added a comment.

@rjmccall thanks for the feedback. I initially kept the arguments optional because the single Location argument used to be optional as well.

But I think we can pass proper locations for all loops constructed by clang and I updated my patch to make the Start and End locations non-optional, which in turn required passing them through when constructing loops in lib/CodeGen/CGStmtOpenMP.cpp .


https://reviews.llvm.org/D25764