[PATCH] D133066: fix a typo in comment of AddConversionCandidate
zhouyizhou via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sat Oct 8 16:02:58 PDT 2022
zhouyizhou added a comment.
After 4 weeks' study, I think the comment didn't need to be changed, sorry to have bring your so much trouble.
During this valuable process of studying, I grow up a lot. I learned to read the C++ standard, and compare the standard to its implementation.
In my case, the "user-defined conversion" is the variable "Candidate", the "second standard conversion sequence" is the object member "Candidate.FinalConversion".
The only pity during my study is that I can't find a example code to let Clang (even with commit cba72b1f620fd) hit the code below above comment.
I am going to close this thread.
Thanks for reviewing my patch, and for your guidance.
Cheers
Zhouyi
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D133066/new/
https://reviews.llvm.org/D133066
More information about the cfe-commits
mailing list