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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
