cdavis5x added a comment.

Are you sure this shouldn't also define `_UNICODE`? //looks// Hmm... GCC 
doesn't define it. I wonder if we should anyway. A user who set `-municode` may 
also be using `<tchar.h>`, and thus may want those macros set to their Unicode 
counterparts.

I actually believe this is supposed to have one other effect: it sets the entry 
point to `wmainCRTStartup()`/`wWinMainCRTStartup()`, making the user entry 
point `wmain()`/`wWinMain()`, which take wide arguments:

  int wmain(int argc, wchar_t **argv);
  int WINAPI wWinMain(HINSTANCE hinst, HINSTANCE hinstPrev, LPWSTR pwszCmdLine, 
int nCmdShow);

But that's a link-time effect. I believe it has no bearing on @rnk's comment.

...Actually, that suggests that we may want to warn if the user defines 
`main()` or `WinMain()` instead of their Unicode counterparts when `-municode` 
is given--which means we may want to pass it to the frontend after all.


https://reviews.llvm.org/D50199



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to