[libc-commits] [libc] [libc] init uefi os target (PR #120687)

Joseph Huber via libc-commits libc-commits at lists.llvm.org
Sat Feb 8 05:15:42 PST 2025


================
@@ -736,6 +738,11 @@ function(add_libc_hermetic test_name)
     endif()
   endforeach()
 
+  if(LIBC_TARGET_OS_IS_UEFI)
+    target_link_options(${fq_build_target_name} PRIVATE
+      ${LIBC_COMPILE_OPTIONS_DEFAULT} "-Wl,/lldmingw")
----------------
jhuber6 wrote:

I feel like that's not the proper solution, though I don't know off the top of my head why symbols would be missing. I'd think that `lld-link` wouldn't have the static library ordering thing that GNU linkers have. IF that's the case then it would be easier to just prepend the linker arguments with `--whole-archive` or whatever the `lld-link` equivalent is. (I think there's a per-static library version we could use when building the test list.)

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


More information about the libc-commits mailing list