diff options
| -rw-r--r-- | llvm/lib/ExecutionEngine/Orc/Core.cpp | 2 | ||||
| -rw-r--r-- | llvm/unittests/ExecutionEngine/Orc/CoreAPIsTest.cpp | 7 |
2 files changed, 8 insertions, 1 deletions
diff --git a/llvm/lib/ExecutionEngine/Orc/Core.cpp b/llvm/lib/ExecutionEngine/Orc/Core.cpp index aeb10a584c3..82be34476f6 100644 --- a/llvm/lib/ExecutionEngine/Orc/Core.cpp +++ b/llvm/lib/ExecutionEngine/Orc/Core.cpp @@ -452,7 +452,7 @@ void VSO::addDependencies(const SymbolFlagsMap &Dependants, if (OtherMI.IsFinalized) transferFinalizedNodeDependencies(MI, Name, OtherMI); - else { + else if (&OtherVSO != this || OtherSymbol != Name) { OtherMI.Dependants[this].insert(Name); DepsOnOtherVSO.insert(OtherSymbol); } diff --git a/llvm/unittests/ExecutionEngine/Orc/CoreAPIsTest.cpp b/llvm/unittests/ExecutionEngine/Orc/CoreAPIsTest.cpp index 01f81d9a80f..c8f30dbfd94 100644 --- a/llvm/unittests/ExecutionEngine/Orc/CoreAPIsTest.cpp +++ b/llvm/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"; |
