[llvm] 222d8fa - [ORC] Try to skip unsupported targets in reoptimization test.

Sunho Kim via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 11 09:49:20 PDT 2024


Author: Sunho Kim
Date: 2024-10-12T01:49:06+09:00
New Revision: 222d8fadb458d9b27c27e3314337ac3771b1d33b

URL: https://github.com/llvm/llvm-project/commit/222d8fadb458d9b27c27e3314337ac3771b1d33b
DIFF: https://github.com/llvm/llvm-project/commit/222d8fadb458d9b27c27e3314337ac3771b1d33b.diff

LOG: [ORC] Try to skip unsupported targets in reoptimization test.

Many build bots are getting failures because of this:
https://lab.llvm.org/buildbot/#/builders/140/builds/8600
https://lab.llvm.org/buildbot/#/builders/137/builds/6824
https://lab.llvm.org/buildbot/#/builders/140/builds/8600

Added: 
    

Modified: 
    llvm/unittests/ExecutionEngine/Orc/JITLinkRedirectionManagerTest.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/unittests/ExecutionEngine/Orc/JITLinkRedirectionManagerTest.cpp b/llvm/unittests/ExecutionEngine/Orc/JITLinkRedirectionManagerTest.cpp
index 170637d78d292c..6d6ebd97554b63 100644
--- a/llvm/unittests/ExecutionEngine/Orc/JITLinkRedirectionManagerTest.cpp
+++ b/llvm/unittests/ExecutionEngine/Orc/JITLinkRedirectionManagerTest.cpp
@@ -32,6 +32,11 @@ class JITLinkRedirectionManagerTest : public testing::Test {
       consumeError(JTMB.takeError());
       GTEST_SKIP();
     }
+    auto DLOrErr = JTMB->getDefaultDataLayoutForTarget();
+    if (!DLOrErr) {
+      consumeError(DLOrErr.takeError());
+      GTEST_SKIP();
+    }
 
     ES = std::make_unique<ExecutionSession>(
         std::make_unique<UnsupportedExecutorProcessControl>(
@@ -39,8 +44,7 @@ class JITLinkRedirectionManagerTest : public testing::Test {
     JD = &ES->createBareJITDylib("main");
     ObjLinkingLayer = std::make_unique<ObjectLinkingLayer>(
         *ES, std::make_unique<InProcessMemoryManager>(16384));
-    DL = std::make_unique<DataLayout>(
-        cantFail(JTMB->getDefaultDataLayoutForTarget()));
+    DL = std::make_unique<DataLayout>(std::move(*DLOrErr));
   }
   JITDylib *JD{nullptr};
   std::unique_ptr<ExecutionSession> ES;


        


More information about the llvm-commits mailing list