[PATCH] D137263: add boundary check for ASTUnresolvedSet::erase

zhouyizhou via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 2 17:54:59 PDT 2022


zhouyizhou added a comment.

In D137263#3903845 <https://reviews.llvm.org/D137263#3903845>, @rjmccall wrote:

> Good catch!  Easy to see how this escaped notice for a decade, but still, it'll be good to fix.
>
> LGTM with a minor improvement.

Thank John for your encouragement and guidance!
I have no write access to LLVM project, could you please commit it for me when you are free?

Cheers
Zhouyi Zhou
zhouzhouyi at gmail.com


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

https://reviews.llvm.org/D137263



More information about the cfe-commits mailing list