[PATCH] Abstract out parts of thunk emission code, add support for simple thunks for -cxx-abi microsoft

Timur Iskhodzhanov timurrrr at google.com
Mon Sep 30 09:43:56 PDT 2013


Hi rnk,

Hi Reid,

This is a preliminary patch that makes clang-cl pass a few sanity tests that require thunks for multiple and/or virtual inheritance.

The patch still fails one test [strange, I think I've fixed that one already] I hope to get the patch that passes "check-clang" tomorrow.

In the meantime, can you please take a look at the general direction? 

--
Timur

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

Files:
  include/clang/Basic/ABI.h
  lib/AST/MicrosoftMangle.cpp
  lib/AST/VTableBuilder.cpp
  lib/CodeGen/CGCXXABI.h
  lib/CodeGen/CGVTables.cpp
  lib/CodeGen/ItaniumCXXABI.cpp
  lib/CodeGen/MicrosoftCXXABI.cpp
  test/CodeGenCXX/destructor-exception-spec.cpp
  test/CodeGenCXX/microsoft-abi-thunks.cpp
  test/CodeGenCXX/microsoft-abi-vtables-virtual-inheritance.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D1787.1.patch
Type: text/x-patch
Size: 17436 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130930/15a73378/attachment.bin>


More information about the cfe-commits mailing list