r227838 - MS ABI: Add more documentation and tests for novtable

David Majnemer david.majnemer at gmail.com
Mon Feb 2 11:05:46 PST 2015


Author: majnemer
Date: Mon Feb  2 13:05:46 2015
New Revision: 227838

URL: http://llvm.org/viewvc/llvm-project?rev=227838&view=rev
Log:
MS ABI: Add more documentation and tests for novtable

Added:
    cfe/trunk/test/CodeGenCXX/ms-novtable.cpp
    cfe/trunk/test/SemaCXX/ms-novtable.cpp
Modified:
    cfe/trunk/include/clang/Basic/Attr.td
    cfe/trunk/include/clang/Basic/AttrDocs.td

Modified: cfe/trunk/include/clang/Basic/Attr.td
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/Attr.td?rev=227838&r1=227837&r2=227838&view=diff
==============================================================================
--- cfe/trunk/include/clang/Basic/Attr.td (original)
+++ cfe/trunk/include/clang/Basic/Attr.td Mon Feb  2 13:05:46 2015
@@ -1764,7 +1764,7 @@ def TypeTagForDatatype : InheritableAttr
 def MsNoVTable : InheritableAttr {
   let Spellings = [Declspec<"novtable">];
   let Subjects = SubjectList<[CXXRecord]>;
-  let Documentation = [Undocumented];
+  let Documentation = [MsNoVTableDocs];
 }
 
 def MsProperty : IgnoredAttr {

Modified: cfe/trunk/include/clang/Basic/AttrDocs.td
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/AttrDocs.td?rev=227838&r1=227837&r2=227838&view=diff
==============================================================================
--- cfe/trunk/include/clang/Basic/AttrDocs.td (original)
+++ cfe/trunk/include/clang/Basic/AttrDocs.td Mon Feb  2 13:05:46 2015
@@ -1253,6 +1253,15 @@ an error:
 }];
 }
 
+def MsNoVTableDocs : Documentation {
+  let Category = DocCatType;
+  let Content = [{
+This attribute can be added to a class declaration or definition to signal to
+the compiler that constructors and destructors will not reference the virtual
+function table.
+  }];
+}
+
 def OptnoneDocs : Documentation {
   let Category = DocCatFunction;
   let Content = [{

Added: cfe/trunk/test/CodeGenCXX/ms-novtable.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CodeGenCXX/ms-novtable.cpp?rev=227838&view=auto
==============================================================================
--- cfe/trunk/test/CodeGenCXX/ms-novtable.cpp (added)
+++ cfe/trunk/test/CodeGenCXX/ms-novtable.cpp Mon Feb  2 13:05:46 2015
@@ -0,0 +1,21 @@
+// RUN: %clang_cc1 -triple i386-pc-win32 %s -emit-llvm -fms-extensions -fms-compatibility -fno-rtti -o - | FileCheck %s
+
+// CHECK-NOT: @"\01??_7C@@6B@"
+
+// CHECK-DAG: @"\01??_7A2@@6B@"
+
+// CHECK-DAG: @"\01??_7B2@@6B@"
+
+// CHECK-NOT: @"\01??_7B1@@6B@"
+
+// CHECK-NOT: @"\01??_7A1@@6B@"
+
+struct __declspec(novtable) A1 {
+  virtual void a();
+} a1;
+struct                      A2 {
+  virtual void a();
+};
+struct __declspec(novtable) B1 : virtual A1 {} b1;
+struct                      B2 : virtual A1 {} b2;
+struct __declspec(novtable) C  : virtual A2 {} c;

Added: cfe/trunk/test/SemaCXX/ms-novtable.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/SemaCXX/ms-novtable.cpp?rev=227838&view=auto
==============================================================================
--- cfe/trunk/test/SemaCXX/ms-novtable.cpp (added)
+++ cfe/trunk/test/SemaCXX/ms-novtable.cpp Mon Feb  2 13:05:46 2015
@@ -0,0 +1,6 @@
+// RUN: %clang_cc1 %s -fsyntax-only -verify -fms-extensions -Wno-microsoft -std=c++11
+
+struct __declspec(novtable) S {};
+enum __declspec(novtable) E {}; // expected-warning{{'novtable' attribute only applies to classes}}
+int __declspec(novtable) I; // expected-warning{{'novtable' attribute only applies to classes}}
+typedef struct T __declspec(novtable) U; // expected-warning{{'novtable' attribute only applies to classes}}





More information about the cfe-commits mailing list