Skip to content

Commit

Permalink
Add sparc support to BoehmGC CMakeLists.txt
Browse files Browse the repository at this point in the history 
ref #99
  • Loading branch information
@Constellation
Constellation committed Jan 10, 2015
1 parent 167541f commit bd37c33
Showing 1 changed file with 12 additions and 0 deletions.
12 changes: 12 additions & 0 deletions iv/lv5/third_party/gc/CMakeLists.txt
View file
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@
#

SET(CMAKE_LEGACY_CYGWIN_WIN32 0) # Remove when CMake >= 2.8.4 is required
ENABLE_LANGUAGE(ASM)

PROJECT(gc)

Expand Down Expand Up @@ -63,6 +64,17 @@ SET(SRC alloc.c reclaim.c allchblk.c misc.c mach_dep.c os_dep.c
new_hblk.c dbg_mlc.c malloc.c stubborn.c dyn_load.c
typd_mlc.c ptr_chck.c gc_cpp.cc mallocx.c checksums.c
thread_local_alloc.c)

#FIXME(Yusuke Suzuki): added
IF(CMAKE_SYSTEM_PROCESSOR MATCHES "^sparc")
IF(CMAKE_SYSTEM_NAME MATCHES "kNetBSD.*|NetBSD.*")
SET(SRC ${SRC} src/sparc_netbsd_mach_dep.S)
ELSE()
SET(SRC ${SRC} src/sparc_mach_dep.S)
ENDIF()
ENDIF()


SET(LIBS)
OPTION(enable_threads "TODO" NO)
IF(enable_threads)
Expand Down

3 comments on commit bd37c33

@syoyo
Copy link

@syoyo syoyo commented on bd37c33 Jan 11, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think its better to move ENABLE_LANGUAGE(ASM) into IF(CMAKE_SYSTEM_PROCESSOR MATCHES "^sparc") clause, since for non SPARC architecture ASM is not required.

@Constellation
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make sense. I'll fix it :)

@Constellation
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done! 08508f7

Please sign in to comment.