[PATCH] D29972: Make Lit tests C++11 compatible - accessible destructors

2017-02-24 Thread Charles Li via Phabricator via cfe-commits
tigerleapgorge abandoned this revision. tigerleapgorge added a comment. https://reviews.llvm.org/D20710 is a superset of this patch. https://reviews.llvm.org/D20710 has been committed in https://reviews.llvm.org/rL296184. No need for this patch. https://reviews.llvm.org/D29972 __

[PATCH] D29972: Make Lit tests C++11 compatible - accessible destructors

2017-02-14 Thread Charles Li via Phabricator via cfe-commits
tigerleapgorge created this revision. Herald added a subscriber: mehdi_amini. I am continuing to make Lit tests C++11 compatible. This patch contains 2 tests previously in https://reviews.llvm.org/D20710. In both tests, I have made the base class destructors “protected” so they are accessible to