[PATCH] D51192: Fix reported range of partial token replacement

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 6 12:59:34 PDT 2018


aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.

In https://reviews.llvm.org/D51192#1226326, @steveire wrote:

> As far as I know, no existing clang-tidy checks are affected. I discovered this while implementing a custom check. See https://bugs.llvm.org/show_bug.cgi?id=38678


It's unfortunate this is going in without tests, but it still LGTM.

> By the way, is there some keyword I should use in commit messages to link to bugs properly?

Add the following to the end of the commit message and I believe it will automatically close the Phab review:

  Differential Revision: <URL>

Where the URL is the URL to this review.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D51192





More information about the cfe-commits mailing list