Skip to content

Conversation

@bulbazord
Copy link
Member

All the unifdef logic was moved to source/API/CMakeLists.txt. This was left behind during that move.

All the unifdef logic was moved to source/API/CMakeLists.txt. This was
left behind during that move.
@llvmbot
Copy link
Member

llvmbot commented Oct 14, 2025

@llvm/pr-subscribers-lldb

Author: Alex Langford (bulbazord)

Changes

All the unifdef logic was moved to source/API/CMakeLists.txt. This was left behind during that move.


Full diff: https://github.com/llvm/llvm-project/pull/163446.diff

1 Files Affected:

  • (modified) lldb/cmake/modules/LLDBFramework.cmake (-2)
diff --git a/lldb/cmake/modules/LLDBFramework.cmake b/lldb/cmake/modules/LLDBFramework.cmake
index c6f00ed05cfc2..23d9d49d0cefc 100644
--- a/lldb/cmake/modules/LLDBFramework.cmake
+++ b/lldb/cmake/modules/LLDBFramework.cmake
@@ -68,8 +68,6 @@ if(NOT APPLE_EMBEDDED)
   )
 endif()
 
-find_program(unifdef_EXECUTABLE unifdef)
-
 # Wrap output in a target, so lldb-framework can depend on it.
 add_custom_target(liblldb-resource-headers DEPENDS lldb-sbapi-dwarf-enums ${lldb_staged_headers})
 set_target_properties(liblldb-resource-headers PROPERTIES FOLDER "LLDB/Resources")

@bulbazord bulbazord merged commit a7f1910 into llvm:main Oct 14, 2025
12 checks passed
@bulbazord bulbazord deleted the small-cmake-cleanup branch October 14, 2025 22:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants