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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to