[PATCH] D53856: [analyzer] Put llvm.Conventions back in alpha

Umann Kristóf via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 30 07:27:37 PDT 2018


Szelethus added a comment.

In https://reviews.llvm.org/D53856#1279887, @NoQ wrote:

> This might be also covered by @rnkovacs's string buffer escape checker - either already or eventually, it'll become just yet another string view API that the checker supports.


I thought about that too, adding some `StringRef` specific information to that checker makes more sense then essentially duplicating the logic. However, what @MTC mentioned about `ArrayRef<T>` would be a neat addition too, and maybe it isn't worth making `InnerPointerChecker` //that// general.

@rnkovacs, any thoughts?


https://reviews.llvm.org/D53856





More information about the cfe-commits mailing list