From 57a0e0d4c2a1c83f8c0d50b893616220d1167e00 Mon Sep 17 00:00:00 2001 From: Stephen Tozer Date: Mon, 8 Mar 2021 15:42:41 +0000 Subject: [PATCH] Fix 2: [DebugInfo] Support DIArgList in DbgVariableIntrinsic Changes to function calls in LocalTest resulted in comparisons between unsigned values and signed literals; the latter have been updated to be unsigned to prevent this warning. --- llvm/unittests/Transforms/Utils/LocalTest.cpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/llvm/unittests/Transforms/Utils/LocalTest.cpp b/llvm/unittests/Transforms/Utils/LocalTest.cpp index f2fce5e..720472d 100644 --- a/llvm/unittests/Transforms/Utils/LocalTest.cpp +++ b/llvm/unittests/Transforms/Utils/LocalTest.cpp @@ -524,7 +524,7 @@ struct SalvageDebugInfoTest : ::testing::Test { } bool doesDebugValueDescribeX(const DbgValueInst &DI) { - if (DI.getNumVariableLocationOps() != 1) + if (DI.getNumVariableLocationOps() != 1u) return false; const auto &CI = *cast(DI.getValue(0)); if (CI.isZero()) @@ -536,7 +536,7 @@ struct SalvageDebugInfoTest : ::testing::Test { } bool doesDebugValueDescribeY(const DbgValueInst &DI) { - if (DI.getNumVariableLocationOps() != 1) + if (DI.getNumVariableLocationOps() != 1u) return false; const auto &CI = *cast(DI.getVariableLocationOp(0)); if (CI.isZero()) @@ -762,14 +762,14 @@ TEST(Local, ReplaceAllDbgUsesWith) { EXPECT_TRUE(replaceAllDbgUsesWith(A, F_, F_, DT)); auto *ADbgVal = cast(A.getNextNode()); - EXPECT_EQ(ADbgVal->getNumVariableLocationOps(), 1); + EXPECT_EQ(ADbgVal->getNumVariableLocationOps(), 1u); EXPECT_EQ(ConstantInt::get(A.getType(), 0), ADbgVal->getVariableLocationOp(0)); // Introduce a use-before-def. Check that the dbg.values for %f become undef. EXPECT_TRUE(replaceAllDbgUsesWith(F_, G, G, DT)); auto *FDbgVal = cast(F_.getNextNode()); - EXPECT_EQ(FDbgVal->getNumVariableLocationOps(), 1); + EXPECT_EQ(FDbgVal->getNumVariableLocationOps(), 1u); EXPECT_TRUE(FDbgVal->isUndef()); SmallVector FDbgVals; -- 2.7.4