[PATCH] Use GEPs correctly when adjusting "this" in MicrosoftCXXABI

Richard Smith richard at metafoo.co.uk
Tue Oct 22 16:52:48 PDT 2013



================
Comment at: lib/CodeGen/MicrosoftCXXABI.cpp:636-638
@@ +635,5 @@
+    if (ML.VBase) {
+      // Non-virtual adjustment might result in a pointer outside the allocated
+      // object, e.g. if the final overrider class is laid out after the virtual
+      // base that declares a method in the most derived class.
+      // FIXME: Update the code that emits this adjustment in thunks prologues.
----------------
How can this give an offset that takes you outside the complete object?


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



More information about the cfe-commits mailing list