From c6679d1e091164751b6922d7ca0ce2c6ebb9473c Mon Sep 17 00:00:00 2001 From: Lang Hames Date: Thu, 14 Jun 2018 15:32:59 +0000 Subject: [ORC] Filter out self-dependencies in VSO::addDependencies. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@334724 91177308-0d34-0410-b5e6-96231b3b80d8 --- unittests/ExecutionEngine/Orc/CoreAPIsTest.cpp | 7 +++++++ 1 file changed, 7 insertions(+) (limited to 'unittests/ExecutionEngine') diff --git a/unittests/ExecutionEngine/Orc/CoreAPIsTest.cpp b/unittests/ExecutionEngine/Orc/CoreAPIsTest.cpp index 01f81d9a80f..c8f30dbfd94 100644 --- a/unittests/ExecutionEngine/Orc/CoreAPIsTest.cpp +++ b/unittests/ExecutionEngine/Orc/CoreAPIsTest.cpp @@ -301,10 +301,17 @@ TEST(CoreAPIsTest, TestCircularDependenceInOneVSO) { EXPECT_TRUE(Unresolved.empty()) << "Failed to resolve \"Baz\""; } + // Add a circular dependency: Foo -> Bar, Bar -> Baz, Baz -> Foo. FooR->addDependencies({{&V, SymbolNameSet({Bar})}}); BarR->addDependencies({{&V, SymbolNameSet({Baz})}}); BazR->addDependencies({{&V, SymbolNameSet({Foo})}}); + // Add self-dependencies for good measure. This tests that the implementation + // of addDependencies filters these out. + FooR->addDependencies({{&V, SymbolNameSet({Foo})}}); + BarR->addDependencies({{&V, SymbolNameSet({Bar})}}); + BazR->addDependencies({{&V, SymbolNameSet({Baz})}}); + EXPECT_FALSE(FooResolved) << "\"Foo\" should not be resolved yet"; EXPECT_FALSE(BarResolved) << "\"Bar\" should not be resolved yet"; EXPECT_FALSE(BazResolved) << "\"Baz\" should not be resolved yet"; -- cgit v1.2.3