Skip to content

Commit 40962a3

Browse files
authored
[build] Update Slang dependency properties before installing (#9017)
The properties of Slang dependencies are updated to avoid installing their headers alongside our headers. However, the new properties are not correctly taken into account if we issue an install command before the update. This commit fixes the update order in the main CMakeLists.txt file.
1 parent 41c6990 commit 40962a3

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

CMakeLists.txt

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -604,12 +604,13 @@ if(CIRCT_SLANG_FRONTEND_ENABLED)
604604
get_target_property(dep_target ${dep_alias} ALIASED_TARGET)
605605
message(STATUS "Installing slang dependency ${dep_target}")
606606
set_property(GLOBAL APPEND PROPERTY CIRCT_EXPORTS ${dep_target})
607-
install(TARGETS ${dep_target} EXPORT CIRCTTargets)
608607

609608
# Disable the installation of headers coming from third-party libraries.
610609
# We won't use those APIs directly. Just make them static libraries for
611610
# the sake of running slang normally.
612611
set_target_properties(${dep_target} PROPERTIES PUBLIC_HEADER "")
612+
613+
install(TARGETS ${dep_target} EXPORT CIRCTTargets)
613614
endif()
614615
endforeach()
615616
set_property(GLOBAL APPEND PROPERTY CIRCT_EXPORTS slang_slang)

0 commit comments

Comments
 (0)