[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