[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