[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