From 00e79db85196dcf8911a04e19ba2960e6bc17216 Mon Sep 17 00:00:00 2001 From: Justin Lebar Date: Wed, 13 Jul 2016 18:27:49 +0000 Subject: [PATCH] Fix warning in ObjectTransformLayerTest. Doing "I++" inside of an EXPECT_* triggers warning: expression with side effects has no effect in an unevaluated context because EXPECT_* partially expands to EqHelper<(sizeof(::testing::internal::IsNullLiteralHelper(MockObjects[I++] + 1)) == 1)> which is an unevaluated context. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@275293 91177308-0d34-0410-b5e6-96231b3b80d8 --- unittests/ExecutionEngine/Orc/ObjectTransformLayerTest.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/unittests/ExecutionEngine/Orc/ObjectTransformLayerTest.cpp b/unittests/ExecutionEngine/Orc/ObjectTransformLayerTest.cpp index 908dba11e29..e1b1f2f9278 100644 --- a/unittests/ExecutionEngine/Orc/ObjectTransformLayerTest.cpp +++ b/unittests/ExecutionEngine/Orc/ObjectTransformLayerTest.cpp @@ -62,7 +62,8 @@ public: EXPECT_EQ(MockResolver, *Resolver) << "Resolver should pass through"; size_t I = 0; for (auto &ObjPtr : Objects) { - EXPECT_EQ(MockObjects[I++] + 1, *ObjPtr) << "Transform should be applied"; + EXPECT_EQ(MockObjects[I] + 1, *ObjPtr) << "Transform should be applied"; + I++; } EXPECT_EQ(MockObjects.size(), I) << "Number of objects should match"; LastCalled = "addObjectSet"; -- 2.50.0