rnk added a comment. Seems reasonable
================ Comment at: clang/lib/Sema/SemaDecl.cpp:11206-11207 + } else if (FT->getCallConv() != CC_X86StdCall) { + // Default calling convention for WinMain, wWinMain and DllMain is + // __stdcall + FT = Context.adjustFunctionType( ---------------- This should only be done on 32-bit x86 platforms. I think i686-window-msvc is the more special case platform here, so I would recommend making the helper something like `isDefaultStdCall`, which is true for i686-windows-msvc, false for mingw and non-x86 triples, and false for main/wmain. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97941/new/ https://reviews.llvm.org/D97941 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits