[llvm] d67f80a - [ORC] Fix more missing LLVMConsumeError calls.
Lang Hames via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 26 21:45:11 PDT 2021
Author: Lang Hames
Date: 2021-04-26T21:44:26-07:00
New Revision: d67f80a24243bdc9a194d194ecedd254a51e9235
URL: https://github.com/llvm/llvm-project/commit/d67f80a24243bdc9a194d194ecedd254a51e9235
DIFF: https://github.com/llvm/llvm-project/commit/d67f80a24243bdc9a194d194ecedd254a51e9235.diff
LOG: [ORC] Fix more missing LLVMConsumeError calls.
Added:
Modified:
llvm/unittests/ExecutionEngine/Orc/OrcCAPITest.cpp
Removed:
################################################################################
diff --git a/llvm/unittests/ExecutionEngine/Orc/OrcCAPITest.cpp b/llvm/unittests/ExecutionEngine/Orc/OrcCAPITest.cpp
index 7441e8698219..ea868fa2a71d 100644
--- a/llvm/unittests/ExecutionEngine/Orc/OrcCAPITest.cpp
+++ b/llvm/unittests/ExecutionEngine/Orc/OrcCAPITest.cpp
@@ -213,7 +213,8 @@ TEST_F(OrcCAPITestBase, MaterializationUnitCreation) {
LLVMOrcMaterializationUnitRef MU = LLVMOrcAbsoluteSymbols(Pairs, 1);
LLVMOrcJITDylibDefine(MainDylib, MU);
LLVMOrcJITTargetAddress OutAddr;
- if (LLVMOrcLLJITLookup(Jit, &OutAddr, "test")) {
+ if (LLVMErrorRef E = LLVMOrcLLJITLookup(Jit, &OutAddr, "test")) {
+ LLVMConsumeError(E);
FAIL() << "Failed to look up \"test\" symbol (triple = "
<< TargetTriple << ")";
}
@@ -231,7 +232,8 @@ TEST_F(OrcCAPITestBase, DefinitionGenerators) {
nullptr);
LLVMOrcJITDylibAddGenerator(MainDylib, Gen);
LLVMOrcJITTargetAddress OutAddr;
- if (LLVMOrcLLJITLookup(Jit, &OutAddr, "test")) {
+ if (LLVMErrorRef E = LLVMOrcLLJITLookup(Jit, &OutAddr, "test")) {
+ LLVMConsumeError(E);
FAIL() << "The DefinitionGenerator did not create symbol \"test\" "
<< "(triple = " << TargetTriple << ")";
}
@@ -253,11 +255,13 @@ TEST_F(OrcCAPITestBase, ResourceTrackerDefinitionLifetime) {
LLVMOrcJITDylibCreateResourceTracker(MainDylib);
LLVMOrcThreadSafeModuleRef TSM = createTestModule();
if (LLVMErrorRef E = LLVMOrcLLJITAddLLVMIRModuleWithRT(Jit, RT, TSM)) {
+ LLVMConsumeError(E);
FAIL() << "Failed to add LLVM IR module to LLJIT (triple = "
<< TargetTriple << ")";
}
LLVMOrcJITTargetAddress TestFnAddr;
- if (LLVMOrcLLJITLookup(Jit, &TestFnAddr, "sum")) {
+ if (LLVMErrorRef E = LLVMOrcLLJITLookup(Jit, &TestFnAddr, "sum")) {
+ LLVMConsumeError(E);
FAIL() << "Symbol \"sum\" was not added into JIT (triple = "
<< TargetTriple << ")";
}
@@ -283,11 +287,13 @@ TEST_F(OrcCAPITestBase, ResourceTrackerTransfer) {
LLVMOrcJITDylibCreateResourceTracker(MainDylib);
LLVMOrcThreadSafeModuleRef TSM = createTestModule();
if (LLVMErrorRef E = LLVMOrcLLJITAddLLVMIRModuleWithRT(Jit, DefaultRT, TSM)) {
+ LLVMConsumeError(E);
FAIL() << "Failed to add LLVM IR module to LLJIT (triple = "
<< TargetTriple << ")";
}
LLVMOrcJITTargetAddress Addr;
- if (LLVMOrcLLJITLookup(Jit, &Addr, "sum")) {
+ if (LLVMErrorRef E = LLVMOrcLLJITLookup(Jit, &Addr, "sum")) {
+ LLVMConsumeError(E);
FAIL() << "Symbol \"sum\" was not added into JIT (triple = "
<< TargetTriple << ")";
}
More information about the llvm-commits
mailing list