rsmith added inline comments.
================
Comment at: lib/CodeGen/CGClass.cpp:2098
@@ -2063,3 +2097,3 @@
// Initialize the vtable pointer for this base.
- InitializeVTablePointer(Base, NearestVBase, OffsetFromNearestVBase,
- VTableClass);
+ VPtr vptr{Base, NearestVBase, OffsetFromNearestVBase, VTableClass};
+ vptrs.push_back(vptr);
----------------
Prazek wrote:
> majnemer wrote:
> > This kind of initialization is not very common in LLVM.
> is It worth adding constructor just to change { with ( ?
No, just add an `=` before the `{`.
http://reviews.llvm.org/D11859
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
