[PATCH] D51884: [clang-tidy] ExprMutationAnalyzer: construct from references. Fixes PR38888

Jonas Toth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 10 12:53:34 PDT 2018


JonasToth added a comment.

I am in favor of the change.

I feel that the `findMutation...` functions that take raw pointers should get the assertions though, at least the ones in the public interface. Having them for the private ones won't hurt either.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D51884





More information about the cfe-commits mailing list