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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
