[llvm] [offload] Standalone build fixes (PR #118173)

Joseph Huber via llvm-commits llvm-commits at lists.llvm.org
Sat Nov 30 09:12:42 PST 2024


=?utf-8?q?Michał_Górny?= <mgorny at gentoo.org>,
=?utf-8?q?Michał_Górny?= <mgorny at gentoo.org>,
=?utf-8?q?Michał_Górny?= <mgorny at gentoo.org>,
=?utf-8?q?Michał_Górny?= <mgorny at gentoo.org>,
=?utf-8?q?Michał_Górny?= <mgorny at gentoo.org>,
=?utf-8?q?Michał_Górny?= <mgorny at gentoo.org>,
=?utf-8?q?Michał_Górny?= <mgorny at gentoo.org>,
=?utf-8?q?Michał_Górny?= <mgorny at gentoo.org>,
=?utf-8?q?Michał_Górny?= <mgorny at gentoo.org>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/118173 at github.com>


================
@@ -7,7 +7,10 @@ add_library(PluginCommon OBJECT
   src/RPC.cpp
   src/Utils/ELF.cpp
 )
-add_dependencies(PluginCommon intrinsics_gen LLVMProfileData)
+add_dependencies(PluginCommon intrinsics_gen)
+if (TARGET LLVMProfileData)
+  add_dependencies(PluginCommon LLVMProfileData)
+endif()
----------------
jhuber6 wrote:

I don't think it needs to be here, it's a library target so it should go with the other LLVM libraries. Moving it should avoid the need for handling object libraries like this.

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


More information about the llvm-commits mailing list