[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