[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