[llvm] [Orc] Add NotifyCreated callback for LLJITBuilder (PR #84175)

Mehdi Amini via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 7 15:20:32 PST 2024


Stefan =?utf-8?q?Gränitz?= <stefan.graenitz at gmail.com>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/84175 at github.com>


================
@@ -474,6 +487,11 @@ class LLJITBuilderSetters {
     std::unique_ptr<JITType> J(new JITType(impl(), Err));
     if (Err)
       return std::move(Err);
+
+    if (impl().NotifyCreated)
+      if (Error Err = impl().NotifyCreated(*J))
+        return Err;
----------------
joker-eph wrote:

I don't know why the premerge CI didn't fail, but everywhere else in this function is `return std::move(Err);`

I made the change in 5d33f7176b00

https://github.com/llvm/llvm-project/pull/84175


More information about the llvm-commits mailing list