[PATCH] D55044: [clang-tidy] check for Abseil make_unique

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 13 05:10:41 PST 2018


lebedev.ri added a comment.

In D55044#1329604 <https://reviews.llvm.org/D55044#1329604>, @hokein wrote:

> In fact, the existing `modernize-make-unique` can be configured to support `absl::make_unique`, you'd just need to configure the check option `MakeSmartPtrFunction` to `absl::make_unique` (this is what we do inside google).


See https://reviews.llvm.org/D52670#change-eVDYJhWSHWeW (the `getModuleOptions()` part) on how to do that

> The biggest missing part of the `modernize-make-unique` is `absl::WrapUnique`, I think we should extend `MakeSmartPtrCheck` class (maybe add hooks) to support it.




CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55044/new/

https://reviews.llvm.org/D55044





More information about the cfe-commits mailing list