Skip to content

Commit

Permalink
build: Remove redundant linking under gcc for test harnesses (#1779)
Browse files Browse the repository at this point in the history
After #1774 the workaround for
the Windows variant of `main` being discarded early by `ld` is no longer
necessary.
  • Loading branch information
jcm93 authored Jan 19, 2025
1 parent 639ebe6 commit da7a79b
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 18 deletions.
9 changes: 0 additions & 9 deletions tests/i8080/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -4,15 +4,6 @@ target_include_directories(i8080 PRIVATE ${CMAKE_SOURCE_DIR})

target_link_libraries(i8080 PRIVATE ares::ares)

if(OS_WINDOWS)
if(CMAKE_CXX_COMPILER_ID STREQUAL "GNU")
target_link_libraries(
i8080
PRIVATE nall
)
endif()
endif()

set_target_properties(i8080 PROPERTIES FOLDER tests PREFIX "")
target_enable_subproject(i8080 "i8080 processor test harness")
set(CONSOLE TRUE)
Expand Down
9 changes: 0 additions & 9 deletions tests/m68000/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -4,15 +4,6 @@ target_include_directories(m68000 PRIVATE ${CMAKE_SOURCE_DIR})

target_link_libraries(m68000 PRIVATE ares::ares)

if(OS_WINDOWS)
if(CMAKE_CXX_COMPILER_ID STREQUAL "GNU")
target_link_libraries(
m68000
PRIVATE nall
)
endif()
endif()

set_target_properties(m68000 PROPERTIES FOLDER tests PREFIX "")
target_enable_subproject(m68000 "m68000 processor test harness")
set(CONSOLE TRUE)
Expand Down

0 comments on commit da7a79b

Please sign in to comment.