njames93 added a comment.

In D117306#3247417 <https://reviews.llvm.org/D117306#3247417>, 
@LegalizeAdulthood wrote:

> In D117306#3245915 <https://reviews.llvm.org/D117306#3245915>, @njames93 
> wrote:
>
>> How does this check play with the `modernize-make-shared` check?
>
> Wouldn't it be orthogonal?
>
> This check looks for existing `make_shared` usages, whereas
> modernize-make-shared adds new `make_shared` usages from
> `new shared_ptr` usages.  I wouldn't expect `modernize-make-shared`
> to generate mismatched scalar/array `shared_ptr` instances.

This check looks for constructions of shared_ptr types from an array new 
expression. modernize-make-shared looks for constructions of shared_ptr types 
using the new expression, However I'm not sure how it handles the array version.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117306

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

Reply via email to