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

Timur Iskhodzhanov timurrrr at google.com
Tue Oct 8 12:19:21 PDT 2013


  Landed as r192220, thanks for the review!


================
Comment at: lib/AST/VTableBuilder.cpp:2255
@@ +2254,3 @@
+struct VTableThunksComparator {
+  bool operator()(const VTableLayout::VTableThunkTy &LHS,
+                  const VTableLayout::VTableThunkTy &RHS) {
----------------
Reid Kleckner wrote:
> Just use a function for this, no need for a class.
Per a discussion via chat: this might be worse performance-wise.


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



More information about the cfe-commits mailing list