[MSVC] Fix check for wchar_t type in case of -fno-wchar

[DmitryPolukhin]

Summary

The example below should work identically with and without compiler native wchar_t support.

void foo(wchar_t * t = L"");

[rnk]

lgtm

test/SemaCXX/no-wchar.cpp

12

Can you add some negative tests like these:

short *p1 = L""; // error
int *p2 = L""; // error

Even if they don't make it into IsStringLiteralToNonConstPointerConversion, it's nice to test around things a bit.

[DmitryPolukhin]

I added few more negative tests. Thank you for the review!

lib/Sema/SemaExprCXX.cpp

if (const PointerType *ToPtrType = ToType->getAs<PointerType>())
             case StringLiteral::UTF16:
             case StringLiteral::UTF32:
               // We don't allow UTF literals to be implicitly converted
               break;
             case StringLiteral::Ascii:
               return (ToPointeeType->getKind() == BuiltinType::Char_U ||
                       ToPointeeType->getKind() == BuiltinType::Char_S);
             case StringLiteral::Wide:
-              return ToPointeeType->isWideCharType();
+              return Context.typesAreCompatible(Context.getWideCharType(),
+                                                QualType(ToPointeeType, 0));
           }
         }
       }
 
   return false;
 }
 
 static ExprResult BuildCXXCastArgument(Sema &S,

test/SemaCXX/no-wchar.cpp

 // RUN: %clang_cc1 -triple i386-pc-win32 -fsyntax-only -fno-wchar -verify %s
 wchar_t x; // expected-error {{unknown type name 'wchar_t'}}
 
 typedef unsigned short wchar_t;
 void foo(const wchar_t* x);
 
 void bar() {
   foo(L"wide string literal");
 }
+
+void foo1(wchar_t * t = L"");
+// expected-warning@-1 {{conversion from string literal to 'wchar_t *' (aka 'unsigned short *') is deprecated}}