[PATCH] [ms-cxxabi] Use x86_cdeclmethodcc for __cdecl methods on win32

Reid Kleckner rnk at google.com
Thu Jan 30 15:38:58 PST 2014


Hi rsmith,

This fixes PR15768, where the sret parameter and the 'this' parameter
are in the wrong order.

Alternatively, I could avoid adding an extra bool to all of these
arrangeLLVMFunctionInfo calls by adding a new convention to the clang
calling convention enum.  We should *never* use it in the AST, but we
could adjust CC_C to CC_CDeclMethod right before calling
arrangeLLVMFunctionInfo.

http://llvm-reviews.chandlerc.com/D2664

Files:
  include/clang/CodeGen/CGFunctionInfo.h
  lib/CodeGen/CGCall.cpp
  lib/CodeGen/CGObjCMac.cpp
  lib/CodeGen/CodeGenABITypes.cpp
  lib/CodeGen/CodeGenTypes.h
  lib/CodeGen/TargetInfo.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D2664.1.patch
Type: text/x-patch
Size: 15814 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140130/db339066/attachment.bin>


More information about the cfe-commits mailing list