[PATCH] D14467: [MS] Fix for bug 25013 - #pragma vtordisp is unknown inside functions.

Reid Kleckner via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 12 15:59:24 PST 2015


rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.

I went ahead and ran some of your test cases through MSVC, and I agree with your results. This is pretty interesting behavior. Thanks for looking into it! Looks good.


================
Comment at: test/SemaCXX/pragma-vtordisp.cpp:35-37
@@ -34,5 +34,5 @@
 struct C {
 // FIXME: Our implementation based on token insertion makes it impossible for
 // the pragma to appear everywhere we should support it.
 //#pragma vtordisp()
   struct D : virtual A {
----------------
With your changes, can we enable this test?


http://reviews.llvm.org/D14467





More information about the cfe-commits mailing list