rnk requested changes to this revision.
rnk added inline comments.
This revision now requires changes to proceed.
================
Comment at: clang/lib/Sema/SemaDeclCXX.cpp:4310-4311
// If no results were found, try to correct typos.
TypoCorrection Corr;
MemInitializerValidatorCCC CCC(ClassDecl);
----------------
We have to make sure our MS compatibility logic fires *before* we try to
correct typos. It's currently a big issue that we don't. You can try adjusting
your test case by adding a similarly named class like `Baze` and see if you
still get the desired behavior.
See this old issue from 2014:
https://github.com/llvm/llvm-project/issues/20623
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D124666/new/
https://reviews.llvm.org/D124666
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
