[PATCH] D11928: Small fixup

Piotr Padlewski via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 11 15:44:36 PDT 2015


Prazek added inline comments.

================
Comment at: include/clang/AST/VTableBuilder.h:54
@@ -53,3 +53,3 @@
 
-  VTableComponent() { }
+  VTableComponent() = default;
 
----------------
rnk wrote:
> Is this ctor used? It leaves Value uninitialized. Maybe we should delete it to ensure that it isn't called, or use a default member initializer for Value below.
it is required, f.e.:

VTableComponents(new VTableComponent[NumVTableComponents]),



http://reviews.llvm.org/D11928





More information about the cfe-commits mailing list