dexonsmith added a subscriber: dexonsmith. dexonsmith added a comment. Great, I should have time to clean this up tomorrow afternoon.
Regarding emplace, this patch as is has tests for emplace, but they depend on r258575 being in tree. You asked me to revert that... I'll wait for your response over in that thread: http://lists.llvm.org/pipermail/cfe-commits/Week-of-Mon-20160118/147795.html I have a few other questions inline below. http://reviews.llvm.org/D16360 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits