[PATCH] D124666: In MSVC compatibility mode, handle unqualified templated base class initialization

[Reid Kleckner via Phabricator via cfe-commits]

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