weliveindetail wrote:
@amy-kwan Thanks for your note and sorry for the long delay. There were so many
unrelated buildbot failures today, that I didn't catch this one. I think it was
fixed meanwhile with
https://github.com/llvm/llvm-project/commit/cb994d41c3afb2bd0b25a4c5b2ac48978bf1b23d
Stefan =?utf-8?q?Gränitz?=
Message-ID:
In-Reply-To:
amy-kwan wrote:
Hi @weliveindetail!
I just wanted to give a heads up, that I believe
13078cbc3eeb0ae91c370ce0f604f7165b26e0c8 is causing a failure of the
clang-ppc64le-rhel bot:
Stefan =?utf-8?q?Gr=C3=A4nitz?=
Message-ID:
In-Reply-To:
jplehr wrote:
I see. Thanks for the explanation and the fix! Bot is back to green.
https://github.com/llvm/llvm-project/pull/84461
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
weliveindetail wrote:
We do need the target and not just the symbols. It could be any non-native
target, but implementing a selection mechanism isn't worth the effort. We just
try ARM and (with my above patch) otherwise skip the test.
https://github.com/llvm/llvm-project/pull/84461
Stefan =?utf-8?q?Gränitz?=
Message-ID:
In-Reply-To:
jplehr wrote:
Thanks! I am quite unfamiliar with that part of the code base and wonder if the
symbol needs to just exist somewhere.
The other thing used there (`InitializeNativeTargetAsmPrinter`) is declared in
`TargetSelect.h`. So, does
weliveindetail wrote:
I will try this
https://github.com/llvm/llvm-project/blob/release/18.x/llvm/unittests/DebugInfo/LogicalView/CodeViewReaderTest.cpp#L482
and push a quick-fix.
https://github.com/llvm/llvm-project/pull/84461
___
cfe-commits
weliveindetail wrote:
Thanks for your note. Looks like the problem is that the ARM target is not
registered. It's an uncommon requirement for a unitttest.. Will see how to
check that at runtime. If you have an idea, let me know. Thanks
https://github.com/llvm/llvm-project/pull/84461
Stefan =?utf-8?q?Gränitz?=
Message-ID:
In-Reply-To:
jplehr wrote:
Hi,
I think this one broke one of our buildbots:
https://lab.llvm.org/buildbot/#/builders/259/builds/1769
I'm happy to help looking into it.
https://github.com/llvm/llvm-project/pull/84461
https://github.com/weliveindetail closed
https://github.com/llvm/llvm-project/pull/84461
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/weliveindetail updated
https://github.com/llvm/llvm-project/pull/84461
From fae2f46d25650b8480f9d3135f33a0d6532f43ca Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Stefan=20Gr=C3=A4nitz?=
Date: Thu, 7 Mar 2024 23:04:22 +0100
Subject: [PATCH 1/2] [clang-repl] Add CreateJITBuilder()
https://github.com/weliveindetail updated
https://github.com/llvm/llvm-project/pull/84461
From fae2f46d25650b8480f9d3135f33a0d6532f43ca Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Stefan=20Gr=C3=A4nitz?=
Date: Thu, 7 Mar 2024 23:04:22 +0100
Subject: [PATCH] [clang-repl] Add CreateJITBuilder() for
https://github.com/vgvassilev approved this pull request.
LGTM!
https://github.com/llvm/llvm-project/pull/84461
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/weliveindetail edited
https://github.com/llvm/llvm-project/pull/84461
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/weliveindetail updated
https://github.com/llvm/llvm-project/pull/84461
From 88271c39b30b84041b4b7fb8b8f34c211d8190d5 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Stefan=20Gr=C3=A4nitz?=
Date: Thu, 7 Mar 2024 23:04:22 +0100
Subject: [PATCH] [clang-repl] Add CreateJITBuilder() for
@@ -102,4 +124,97 @@ TEST(InterpreterExtensionsTest, FindRuntimeInterface) {
EXPECT_EQ(1U, Interp.RuntimeIBPtr->TransformerQueries);
}
+class CustomJBInterpreter : public Interpreter {
+ using CustomJITBuilderCreatorFunction =
+
std::function>()>;
+
@@ -102,4 +124,97 @@ TEST(InterpreterExtensionsTest, FindRuntimeInterface) {
EXPECT_EQ(1U, Interp.RuntimeIBPtr->TransformerQueries);
}
+class CustomJBInterpreter : public Interpreter {
+ using CustomJITBuilderCreatorFunction =
+
std::function>()>;
+
https://github.com/weliveindetail edited
https://github.com/llvm/llvm-project/pull/84461
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/weliveindetail commented:
This PR depends on two predecessor patches, but I wanted to share it for review
already. For the moment please refer to the single commit in
https://github.com/llvm/llvm-project/pull/84461/commits/cc46034e5288ba54dfc8a0ea7d54792cbb99227d
for
@@ -373,21 +373,32 @@ Interpreter::Parse(llvm::StringRef Code) {
static llvm::Expected
createJITTargetMachineBuilder(const std::string ) {
if (TT == llvm::sys::getProcessTriple())
+// This fails immediately if the target backend is not registered
return
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?=
Message-ID:
In-Reply-To:
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Stefan Gränitz (weliveindetail)
Changes
The LLJITBuilder interface provides a very convenient way to configure the
ORCv2 JIT engine. IncrementalExecutor
https://github.com/weliveindetail created
https://github.com/llvm/llvm-project/pull/84461
The LLJITBuilder interface provides a very convenient way to configure the
ORCv2 JIT engine. IncrementalExecutor already used it internally to construct
the JIT, but didn't provide external access. This
21 matches
Mail list logo
