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

Reply via email to