labath added inline comments. ================ Comment at: unittests/Utility/ModuleCacheTest.cpp:74 @@ +73,3 @@ +{ + HostInfo::Initialize(); + ObjectFileELF::Initialize(); ---------------- ovyalov wrote: > If it's one-off initialization can we use setUpTestCase for these purposes > (for HostInfo::Initialize and ObjectFileELF::Initialize)? Good point. I have moved this to the static method (including the variable initialization below, as they are not modified). I have also added the corresponding tearDown method.
Repository: rL LLVM http://reviews.llvm.org/D20570 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits