[MERGE #1134] cross-platform: ChakraCore on OSX

Merge pull request #1134 from obastemur:osx_support
Disclaimer: Currently, ChakraCore is not officially supported on OSX.

This PR targets the individuals interested with experimenting ChakraCore on
OSX. After seeing #1131, spent couple of hours to fix the compile, and link
time issues on OSX target.

Steps to compile :
```
brew install icu4c
./build.sh --debug -j:2 --icu=/usr/local/Cellar/icu4c/54.1/include/
```

Status:
CMAKE has an issue with shared bundles on OSX (a.k.a. so) and currently CH does not
play well with dylib.

Author: obastemur

CMakeLists.txt

@@ -19,7 +19,10 @@ if(CMAKE_SYSTEM_NAME STREQUAL Linux)
     endif()
     set(CLR_CMAKE_PLATFORM_LINUX 1)
 elseif(CMAKE_SYSTEM_NAME STREQUAL Darwin)
-    add_definitions(-DPLATFORM_UNIX)
+    add_definitions(
+        -DPLATFORM_UNIX
+        -DU_DISABLE_RENAMING=1 #in case we link against to an older binary of icu 
+    )
 
     set(CLR_CMAKE_PLATFORM_UNIX 1)
     if(CMAKE_SYSTEM_PROCESSOR STREQUAL x86_64)
@@ -76,6 +79,11 @@ if(CLR_CMAKE_PLATFORM_UNIX)
         -Wno-implicit-function-declaration"
     )
 
+    # todo: fix general visibility of the interface
+    # do not set to `fvisibility=hidden` as it is going to 
+    # prevent the required interface is being exported
+    # clang by default sets fvisibility=default
+
     # CXX WARNING FLAGS
     set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} \
         -Werror\

bin/ChakraCore/CMakeLists.txt

@@ -3,10 +3,6 @@ add_library (ChakraCore SHARED
     ConfigParserExternals.cpp
     TestHooks.cpp
 )
-
-# By default, don't export any symbols from this library
-# We'll manually export the relevant individual functions
-set_target_properties(ChakraCore PROPERTIES CXX_VISIBILITY_PRESET hidden)
 
 target_include_directories (
   ChakraCore PUBLIC
@@ -32,42 +28,88 @@ target_include_directories (
 #  dl: For shared library loading related functions
 #  icuuc: For the ICU (xplat-todo: Make this optional)
 #
-target_link_libraries(ChakraCore
-  -Wl,--no-undefined
-  -Wl,--start-group
-  -Wl,--whole-archive
-  Chakra.Jsrt
-  Chakra.Jsrt.Core
-  -Wl,--no-whole-archive
-  Chakra.Runtime.Types
-  Chakra.Runtime.Math
-  Chakra.Runtime.Library
-  Chakra.Runtime.Language
-  Chakra.Runtime.Debug
-  Chakra.Runtime.ByteCode
-  Chakra.Runtime.PlatformAgnostic
-  Chakra.Runtime.Base
-  Chakra.Parser
-  Chakra.Common.Util
-  Chakra.Common.Memory
-  Chakra.Common.Common
-  Chakra.Common.Core
-  Chakra.Common.DataStructures
-  Chakra.Common.Exceptions
-  Chakra.Common.Codex
-  -Wl,--end-group
-  Chakra.Pal
-  pthread
-  stdc++
-  dl
-  icuuc
-  -Wl,--version-script=${CMAKE_CURRENT_SOURCE_DIR}/libChakraCoreLib.version
-)
+if(CMAKE_SYSTEM_NAME STREQUAL Linux)
+    target_link_libraries(ChakraCore
+      -Wl,--no-undefined
+      -Wl,--start-group
+      -Wl,--whole-archive
+      Chakra.Jsrt
+      Chakra.Jsrt.Core
+      -Wl,--no-whole-archive
+      Chakra.Runtime.Types
+      Chakra.Runtime.Math
+      Chakra.Runtime.Library
+      Chakra.Runtime.Language
+      Chakra.Runtime.Debug
+      Chakra.Runtime.ByteCode
+      Chakra.Runtime.PlatformAgnostic
+      Chakra.Runtime.Base
+      Chakra.Parser
+      Chakra.Common.Util
+      Chakra.Common.Memory
+      Chakra.Common.Common
+      Chakra.Common.Core
+      Chakra.Common.DataStructures
+      Chakra.Common.Exceptions
+      Chakra.Common.Codex
+      -Wl,--end-group
+      Chakra.Pal
+      pthread
+      stdc++
+      dl
+      icuuc
+      -Wl,--version-script=${CMAKE_CURRENT_SOURCE_DIR}/libChakraCoreLib.version
+    )
+    #
+    # Post build step to copy the built shared library
+    # to BuildLinux (or whatever the CMakeBuildDir is)
+    add_custom_command(TARGET ChakraCore POST_BUILD
+      COMMAND ${CMAKE_COMMAND} -E copy_if_different
+      "${CHAKRACORE_BINARY_DIR}/bin/ChakraCore/libChakraCore.so"
+       ${CHAKRACORE_BINARY_DIR}/)
+elseif(CMAKE_SYSTEM_NAME STREQUAL Darwin)
+    # todo: combine link libraries into a list
+    # reminder: order of the link libraries matters
+    # we miss multiple keywords below
+    # can we handle the creation of the list
+    # with a single IF/case per platform?
 
-#
-# Post build step to copy the built shared library
-# to BuildLinux (or whatever the CMakeBuildDir is)
-add_custom_command(TARGET ChakraCore POST_BUILD
-  COMMAND ${CMAKE_COMMAND} -E copy_if_different 
-  "${CHAKRACORE_BINARY_DIR}/bin/ChakraCore/libChakraCore.so"
-   ${CHAKRACORE_BINARY_DIR}/)
+    target_link_libraries(ChakraCore
+      -Wl,-undefined,error
+    #   -Wl,--start-group
+    #   -Wl,--whole-archive
+      Chakra.Jsrt
+      Chakra.Jsrt.Core
+    #   -Wl,--no-whole-archive
+      Chakra.Runtime.Types
+      Chakra.Runtime.Math
+      Chakra.Runtime.Library
+      Chakra.Runtime.Language
+      Chakra.Runtime.Debug
+      Chakra.Runtime.ByteCode
+      Chakra.Runtime.PlatformAgnostic
+      Chakra.Runtime.Base
+      Chakra.Parser
+      Chakra.Common.Util
+      Chakra.Common.Memory
+      Chakra.Common.Common
+      Chakra.Common.Core
+      Chakra.Common.DataStructures
+      Chakra.Common.Exceptions
+      Chakra.Common.Codex
+    #   -Wl,--end-group
+      Chakra.Pal
+      pthread
+      stdc++
+      dl
+      icucore
+    #   -Wl,--version-script=${CMAKE_CURRENT_SOURCE_DIR}/libChakraCoreLib.version
+    )
+    #
+    # Post build step to copy the built shared library
+    # to BuildLinux (or whatever the CMakeBuildDir is)
+    add_custom_command(TARGET ChakraCore POST_BUILD
+      COMMAND ${CMAKE_COMMAND} -E copy_if_different 
+      "${CHAKRACORE_BINARY_DIR}/bin/ChakraCore/libChakraCore.dylib"
+       ${CHAKRACORE_BINARY_DIR}/)
+endif()

bin/ch/CMakeLists.txt

@@ -22,15 +22,23 @@ target_include_directories (ch
   ../../lib/Jsrt
   )
 
-set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} -fPIE -pie")
-
-target_link_libraries (ch
-  PRIVATE Chakra.Pal
-  PRIVATE Chakra.Common.Codex
-  PRIVATE Chakra.Runtime.PlatformAgnostic
-  -Wl,--version-script=${CMAKE_CURRENT_SOURCE_DIR}/ch.version
-  )
+set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} -fPIE")
 
+if(CMAKE_SYSTEM_NAME STREQUAL Linux)
+    set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} -pie") # osx clang adds this by default
+    target_link_libraries (ch
+      PRIVATE Chakra.Pal
+      PRIVATE Chakra.Common.Codex
+      PRIVATE Chakra.Runtime.PlatformAgnostic
+      -Wl,--version-script=${CMAKE_CURRENT_SOURCE_DIR}/ch.version
+      )
+elseif(CMAKE_SYSTEM_NAME STREQUAL Darwin)
+    target_link_libraries (ch
+      PRIVATE Chakra.Pal
+      PRIVATE Chakra.Common.Codex
+      PRIVATE Chakra.Runtime.PlatformAgnostic
+      )
+endif()
 # Add a post build event to the ch target
 # which executes a command to copy ch to
 # BuildLinux for convenience

bin/ch/ChakraRtInterface.cpp

@@ -8,8 +8,12 @@
 LPCSTR chakraDllName = "chakracore.dll";
 #else
 #include <dlfcn.h>
+#ifdef __APPLE__
+LPCSTR chakraDllName = "libChakraCore.dylib";
+#else
 LPCSTR chakraDllName = "libChakraCore.so";
 #endif
+#endif
 
 bool ChakraRTInterface::m_testHooksSetup = false;
 bool ChakraRTInterface::m_testHooksInitialized = false;

build.sh

@@ -176,7 +176,7 @@ if [[ ${#_CXX} > 0 ]]; then
     CC_PREFIX="-DCMAKE_CXX_COMPILER=$_CXX -DCMAKE_C_COMPILER=$_CC"
 fi
 
-build_directory="$CHAKRACORE_DIR/BuildLinux/${BUILD_TYPE,,}"
+build_directory="$CHAKRACORE_DIR/BuildLinux/${BUILD_TYPE:0}"
 if [ ! -d "$build_directory" ]; then
     SAFE_RUN `mkdir -p $build_directory`
 fi

lib/Runtime/Language/amd64/JavascriptOperatorsA.S

@@ -5,9 +5,9 @@
 .intel_syntax noprefix
 #include "unixasmmacros.inc"
 
+// expected fvisibility=default
 .global C_FUNC(amd64_CallWithFakeFrame)
 .global C_FUNC(amd64_ReturnFromCallWithFakeFrame)
-
         // r8       = arg0.
         // rcx      = args size.
         // rdx      = spill size.
@@ -22,7 +22,7 @@ C_FUNC(amd64_CallWithFakeFrame):
         // The stack walker uses this marker to skip this frame.
         // Note that we use rip-relative addressing here since this
         // library is shared and therefore the code has to be position independent
-        lea rax, [rip+C_FUNC(amd64_ReturnFromCallWithFakeFrame)]
+        lea rax, [rip + C_FUNC(amd64_ReturnFromCallWithFakeFrame)]
         mov [rsp+8h], rax
 
         mov rax, [rsp + 28h]
@@ -57,9 +57,9 @@ C_FUNC(amd64_CallWithFakeFrame):
 
         jmp rdi
 
-
         // rcx = args size.
         // rdx = spill size.
+
 .balign 16
 C_FUNC(amd64_ReturnFromCallWithFakeFrame):
         add  rsp, rcx

lib/Runtime/Library/amd64/JavascriptFunctionA.S

@@ -6,13 +6,13 @@
 .intel_syntax noprefix
 #include "unixasmmacros.inc"
 
-.extern C_FUNC(_ZN2Js18JavascriptFunction13DeferredParseEPPNS_14ScriptFunctionE)
-.extern C_FUNC(_ZN2Js18JavascriptFunction19DeferredDeserializeEPNS_14ScriptFunctionE)
-
 .global C_FUNC(_ZN2Js18JavascriptFunction20DeferredParsingThunkEPNS_16RecyclableObjectENS_8CallInfoEz)
 .global C_FUNC(_ZN2Js18JavascriptFunction24DeferredDeserializeThunkEPNS_16RecyclableObjectENS_8CallInfoEz)
 
 #ifndef __APPLE__
+.extern _ZN2Js18JavascriptFunction13DeferredParseEPPNS_14ScriptFunctionE
+.extern _ZN2Js18JavascriptFunction19DeferredDeserializeEPNS_14ScriptFunctionE
+
 .type _ZN2Js18JavascriptFunction20DeferredParsingThunkEPNS_16RecyclableObjectENS_8CallInfoEz, @function
 .type _ZN2Js18JavascriptFunction24DeferredDeserializeThunkEPNS_16RecyclableObjectENS_8CallInfoEz, @function
 #endif
@@ -103,11 +103,12 @@ LOCAL_LABEL(function_done):
         lea rsp, [rbp]
         pop_nonvol_reg rbp
         ret
+
 NESTED_END amd64_CallFunction, _TEXT
 
 
-// extrn ?DeferredParse@JavascriptFunction@Js@@SAP6APEAXPEAVRecyclableObject@2@UCallInfo@2@ZZPEAPEAVScriptFunction@2@@Z : PROC
 .balign 16
+.text
 C_FUNC(_ZN2Js18JavascriptFunction20DeferredParsingThunkEPNS_16RecyclableObjectENS_8CallInfoEz):
         push rbp
         lea  rbp, [rsp]
@@ -120,14 +121,15 @@ C_FUNC(_ZN2Js18JavascriptFunction20DeferredParsingThunkEPNS_16RecyclableObjectEN
         //  JavascriptMethod JavascriptFunction::DeferredParse(ScriptFunction**)
         //
         lea rdi, [rbp + 10h]    // &function, setup by custom calling convention
-        call C_PLTFUNC(_ZN2Js18JavascriptFunction13DeferredParseEPPNS_14ScriptFunctionE)
+        call C_FUNC(_ZN2Js18JavascriptFunction13DeferredParseEPPNS_14ScriptFunctionE)
 
         pop rsi
         pop rdi
         pop rbp
 
         jmp rax
 
+
 // Var JavascriptFunction::DeferredDeserializeThunk(
 //              RecyclableObject* function, CallInfo callInfo, ...)
 .balign 16
@@ -144,7 +146,7 @@ C_FUNC(_ZN2Js18JavascriptFunction24DeferredDeserializeThunkEPNS_16RecyclableObje
         //                              ScriptFunction* function)
         //
         //      RDI == function, setup by custom calling convention
-        call C_PLTFUNC(_ZN2Js18JavascriptFunction19DeferredDeserializeEPNS_14ScriptFunctionE)
+        call C_FUNC(_ZN2Js18JavascriptFunction19DeferredDeserializeEPNS_14ScriptFunctionE)
 
         pop rsi
         pop rdi

lib/Runtime/PlatformAgnostic/Platform/CMakeLists.txt

@@ -1,7 +1,5 @@
 project(CHAKRA_PLATFORM_AGNOSTIC)
 
-check_function_exists(gmtime_r HAVE_GMTIME_R)
-
 if(CMAKE_SYSTEM_NAME STREQUAL Linux)
 add_library (Chakra.Runtime.PlatformAgnostic
   Linux/UnicodeText.ICU.cpp
@@ -13,14 +11,15 @@ add_library (Chakra.Runtime.PlatformAgnostic
   )
 elseif(CMAKE_SYSTEM_NAME STREQUAL Darwin)
 add_library (Chakra.Runtime.PlatformAgnostic
-  #Linux/UnicodeText.ICU.cpp
-  #Linux/DateTime.cpp
-  #Linux/HiResTimer.cpp
+  # xplat-todo: implement them for unix! or carry them into common
+  Linux/UnicodeText.ICU.cpp
+  Linux/DateTime.cpp
+  Linux/HiResTimer.cpp
+  Linux/NumbersUtility.cpp
   Unix/SystemInfo.cpp
   Common/UnicodeText.Common.cpp
   )
 endif()
-
 
 target_include_directories (
     Chakra.Runtime.PlatformAgnostic PUBLIC ${CMAKE_CURRENT_SOURCE_DIR})

pal/inc/pal.h

@@ -68,7 +68,6 @@ extern "C" {
 
 #endif
 
-#include <pal_char16.h>
 #include <pal_error.h>
 #include <pal_mstypes.h>