[PATCH] D50347: Add getBeginLoc API to replace getStartLoc

2018-08-09 Thread Stephen Kelly via Phabricator via cfe-commits
steveire added a comment. Accepted as part of a whole-branch review here: http://clang-developers.42468.n3.nabble.com/getLocStart-versus-getStartLoc-td4061010.html Repository: rC Clang https://reviews.llvm.org/D50347 ___ cfe-commits mailing list

[PATCH] D50347: Add getBeginLoc API to replace getStartLoc

2018-08-09 Thread Stephen Kelly via Phabricator via cfe-commits
This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit rL339373: Add getBeginLoc API to replace getStartLoc (authored by steveire, committed by ). Herald added a subscriber: llvm-

[PATCH] D50347: Add getBeginLoc API to replace getStartLoc

2018-08-09 Thread Stephen Kelly via Phabricator via cfe-commits
This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit rC339373: Add getBeginLoc API to replace getStartLoc (authored by steveire, committed by ). Changed prior to commit: http

[PATCH] D50347: Add getBeginLoc API to replace getStartLoc

2018-08-06 Thread Raphael Isemann via Phabricator via cfe-commits
teemperor resigned from this revision. teemperor added a comment. This revision now requires review to proceed. Woops, I wanted to resign actually (and not accept this). Repository: rC Clang https://reviews.llvm.org/D50347 ___ cfe-commits mailing

[PATCH] D50347: Add getBeginLoc API to replace getStartLoc

2018-08-06 Thread Stephen Kelly via Phabricator via cfe-commits
steveire created this revision. Herald added a reviewer: teemperor. Herald added a subscriber: cfe-commits. Repository: rC Clang https://reviews.llvm.org/D50347 Files: include/clang/AST/ExprCXX.h include/clang/AST/Stmt.h include/clang/Analysis/CloneDetection.h lib/CodeGen/CoverageMappi