[PATCH] Fix PR17738 - add support for vtordisp thunks when using -cxx-abi microsoft

Timur Iskhodzhanov timurrrr at google.com
Thu Oct 31 08:37:56 PDT 2013


Hi rnk,

Hi Reid,

Can you please review this patch?

FTR, I've replaced int64_t with uint32_t in MicrosoftMangle as I've never seen anything longer than 8-hex mangling for a number in this ABI and using uint32_t is so much more convenient.

--
Timur

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

Files:
  include/clang/Basic/ABI.h
  lib/AST/ItaniumMangle.cpp
  lib/AST/MicrosoftMangle.cpp
  lib/AST/VTableBuilder.cpp
  lib/CodeGen/ItaniumCXXABI.cpp
  lib/CodeGen/MicrosoftCXXABI.cpp
  test/CodeGenCXX/microsoft-abi-thunks.cpp
  test/CodeGenCXX/microsoft-abi-virtual-inheritance-vtordisps.cpp
  test/CodeGenCXX/microsoft-abi-vtables-virtual-inheritance-vtordisps.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D2079.1.patch
Type: text/x-patch
Size: 32536 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20131031/48b94a4f/attachment.bin>


More information about the cfe-commits mailing list