commit c71a8b0df7c0fd915f5314fd17148f624c798934
author Than McIntosh <thanm@google.com> Tue Jul 17 09:33:45 2018 -0400
committer Than McIntosh <thanm@google.com> Mon Jul 23 15:08:52 2018 +0000
tree 4a78e78ce27936a7541bec289db42250e3600193
parent eb471e8ad75b5513b2f5d123e28e5fd41abf6b53

gollvm: minor unit test fixes

Fix a couple of small buglets in the unit test code.

Change-Id: Ibb9e83b12e6e227175069b284e2c4ab13b136b5d
Reviewed-on: https://go-review.googlesource.com/124376
Reviewed-by: Cherry Zhang <cherryyz@google.com>

unittests/BackendCore/BackendVarTests.cpp

diff --git a/unittests/BackendCore/BackendVarTests.cpp b/unittests/BackendCore/BackendVarTests.cpp
index 60cebd1..237b1cf 100644
--- a/unittests/BackendCore/BackendVarTests.cpp
+++ b/unittests/BackendCore/BackendVarTests.cpp
@@ -725,7 +725,7 @@
   // Expect to see only one lifetime start, since A) loc1 is at the top
   // level, and B) loc2 uses loc1 as declVar.
   const char *expected = "@llvm.lifetime.start.p0i8(i64";
-  EXPECT_EQ(h.countInstancesInModuleDump(expected), 1);
+  EXPECT_EQ(h.countInstancesInModuleDump(expected), 1u);
 }
 
 }

unittests/DriverUtils/DriverUtilsTests.cpp

diff --git a/unittests/DriverUtils/DriverUtilsTests.cpp b/unittests/DriverUtils/DriverUtilsTests.cpp
index f5bc815..643e8a8 100644
--- a/unittests/DriverUtils/DriverUtilsTests.cpp
+++ b/unittests/DriverUtils/DriverUtilsTests.cpp
@@ -264,7 +264,7 @@
       : triple_(trip),
         fs_(fileSystemContents),
         sysroot_(sysroot),
-        detector_(triple_, sysroot_, fs_)
+        detector_(triple_, "", sysroot_, fs_)
   {
     detector_.init();
   }