Szelethus added a comment. In D108753#2988632 <https://reviews.llvm.org/D108753#2988632>, @NoQ wrote:
> This looks good! > > I guess one way to make this //even more// conservative would be to match the > variable inside the delete-expression to the one we expect to get deallocated. Yeah, I thought about that, but one counter argument to that is that once an intent to do some deallocation is already there, its possible that the wrong variable was deleted, or the deallocation wasn't thorough enough. Thanks for the reviews! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D108753/new/ https://reviews.llvm.org/D108753 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
