Revert "Try to use a linker script to export symbols from shared library"

This reverts commit 40e0d54.

Author: mcbarton

.github/workflows/MacOS.yml

@@ -389,7 +389,8 @@ jobs:
                 -DLLVM_ENABLE_WERROR=On                      \
                 ../
         fi
-        cmake --build . --target check-cppinterop --parallel ${{ env.ncpus }}
+        cmake --build . --target check-cppinterop --parallel ${{ env.ncpus }} || true
+        cat /Users/runner/work/CppInterOp/CppInterOp/build/unittests/googletest-prefix/src/googletest-stamp/googletest-build-*.log
         cd ..
         # We need CB_PYTHON_DIR later
         echo "CB_PYTHON_DIR=$CB_PYTHON_DIR" >> $GITHUB_ENV

lib/Interpreter/CMakeLists.txt

@@ -115,9 +115,11 @@ endif()
   )
 
 if(EMSCRIPTEN)
-  # Specify the path to your linker script
-  set(LINKER_SCRIPT_PATH "${CMAKE_SOURCE_DIR}/lib/Interpreter/exports.ld")
+  # Read the undefined symbols from the text file
+  file(READ "${CMAKE_SOURCE_DIR}/lib/Interpreter/undefined_symbols.txt" SYMBOLS_LIST)
 
+  # Replace newlines with spaces
+  string(REPLACE "\n" " " SYMBOLS_LIST "${SYMBOLS_LIST}")
   #FIXME: Setting no_soname=1 is needed until https://github.com/emscripten-core/emscripten/blob/ac676d5e437525d15df5fd46bc2c208ec6d376a3/cmake/Modules/Platform/Emscripten.cmake#L36
   # is patched out of emsdk, as --soname is not recognised by emscripten. A PR to do this has been done here https://github.com/emscripten-core/emscripten/pull/23453 
   #FIXME: A patch is needed to llvm to remove -Wl,-z,defs since it is now recognised on emscripten. What needs to be removed is here
@@ -126,14 +128,10 @@ if(EMSCRIPTEN)
   set_target_properties(clangCppInterOp PROPERTIES 
     NO_SONAME 1
     COMPILE_FLAGS "-s SIDE_MODULE=1"
-    LINK_FLAGS "-s WASM_BIGINT  -s SIDE_MODULE=1"
+    LINK_FLAGS "-s WASM_BIGINT  -s SIDE_MODULE=1 ${SYMBOLS_LIST}"
     SUFFIX ".wasm"
   )
 
-  target_link_options(clangCppInterOp PRIVATE
-    -T${LINKER_SCRIPT_PATH} 
-  )
-  
   add_custom_command(TARGET clangCppInterOp POST_BUILD
     COMMAND ${CMAKE_COMMAND} -E copy $<TARGET_FILE:clangCppInterOp> ${CMAKE_BINARY_DIR}/unittests/CppInterOp/
   )

lib/Interpreter/exports.ld

@@ -0,0 +1,62 @@
+-Wl,--export=_ZN4llvm3sys2fs17getMainExecutableEPKcPv 
+-Wl,--export=_ZN4llvm3sys4path11parent_pathENS_9StringRefENS1_5StyleE 
+-Wl,--export=_ZN3Cpp11GetOperatorEPvNS_8OperatorERNSt3__26vectorIS0_NS2_9allocatorIS0_EEEENS_13OperatorArityE
+-Wl,--export=_ZN4llvm11raw_ostream14flush_nonemptyEv
+-Wl,--export=_ZN4llvm11raw_ostream16SetBufferAndModeEPcmNS0_10BufferKindE
+-Wl,--export=_ZN4llvm11raw_ostream5writeEPKcm
+-Wl,--export=_ZN4llvm11raw_ostreamD2Ev
+-Wl,--export=_ZN4llvm13StringMapImpl11RehashTableEj
+-Wl,--export=_ZN4llvm13StringMapImpl15LookupBucketForENS_9StringRefEj
+-Wl,--export=_ZN4llvm13StringMapImpl4hashENS_9StringRefE
+-Wl,--export=_ZN4llvm15SmallVectorBaseIjE8grow_podEPvmm
+-Wl,--export=_ZN4llvm15allocate_bufferEmm
+-Wl,--export=_ZN4llvm21logAllUnhandledErrorsENS_5ErrorERNS_11raw_ostreamENS_5TwineE
+-Wl,--export=_ZN4llvm25llvm_unreachable_internalEPKcS1_j
+-Wl,--export=_ZN4llvm3sys17RunningOnValgrindEv
+-Wl,--export=_ZN4llvm3sys4path6appendERNS_15SmallVectorImplIcEERKNS_5TwineES7_S7_S7_
+-Wl,--export=_ZN4llvm4dbgsEv
+-Wl,--export=_ZN4llvm4errsEv
+-Wl,--export=_ZN4llvm5APIntC1EjNS_8ArrayRefIyEE
+-Wl,--export=_ZN5clang10ASTContext19createMangleContextEPKNS_10TargetInfoE
+-Wl,--export=_ZN5clang11DeclContext7classofEPKNS_4DeclE
+-Wl,--export=_ZN5clang11Interpreter19getCompilerInstanceEv
+-Wl,--export=_ZN5clang11Interpreter5ParseEN4llvm9StringRefE
+-Wl,--export=_ZN5clang11Interpreter6createENSt3__210unique_ptrINS_16CompilerInstanceENS1_14default_deleteIS3_EEEE
+-Wl,--export=_ZN5clang11Interpreter7ExecuteERNS_22PartialTranslationUnitE
+-Wl,--export=_ZNK5clang4Sema15getStdNamespaceEv
+-Wl,--export=_ZNK5clang4Type27getUnqualifiedDesugaredTypeEv
+-Wl,--export=_ZNK5clang7TagType7getDeclEv
+-Wl,--export=_ZNK5clang7VarDecl28isThisDeclarationADefinitionERNS_10ASTContextE
+-Wl,--export=_ZTVN4llvm18raw_string_ostreamE
+-Wl,--export=clang_Interpreter_dispose
+-Wl,--export=clang_Interpreter_getClangInterpreter
+-Wl,--export=clang_Interpreter_takeInterpreterAsPtr
+-Wl,--export=clang_createInterpreterFromRawPtr
+-Wl,--export=clang_getComplexType
+-Wl,--export=clang_getTypeAsString
+-Wl,--export=clang_instantiateTemplate
+-Wl,--export=_ZN5clang13MangleContext10mangleNameENS_10GlobalDeclERN4llvm11raw_ostreamE
+-Wl,--export=_ZN5clang13MangleContext20shouldMangleDeclNameEPKNS_9NamedDeclE
+-Wl,--export=_ZN5clang26IncrementalCompilerBuilder9CreateCppEv
+-Wl,--export=_ZN5clang4Decl17castToDeclContextEPKS0_
+-Wl,--export=_ZNK3Cpp7JitCall17AreArgumentsValidEPvNS0_7ArgListES1_
+-Wl,--export=_ZNK3Cpp7JitCall17ReportInvokeStartEPvNS0_7ArgListES1_
+-Wl,--export=_ZNK3Cpp7JitCall17ReportInvokeStartEPvmi
+-Wl,--export=_ZNK4llvm5APInt13compareSignedERKS0_
+-Wl,--export=_ZNK4llvm5APInt4sextEj
+-Wl,--export=_ZNK4llvm5APInt4zextEj
+-Wl,--export=_ZNK4llvm5APInt7compareERKS0_
+-Wl,--export=_ZNK4llvm5Error19fatalUncheckedErrorEv
+-Wl,--export=_ZNK5clang10ASTContext14getComplexTypeENS_8QualTypeE
+-Wl,--export=_ZNK5clang10ASTContext19getTypeDeclTypeSlowEPKNS_8TypeDeclE
+-Wl,--export=_ZNK5clang10RecordDecl19isInjectedClassNameEv
+-Wl,--export=_ZNK5clang11DeclContext11decls_beginEv
+-Wl,--export=_ZNK5clang11DeclContext6lookupENS_15DeclarationNameE
+-Wl,--export=_ZNK5clang12FunctionDecl29getTemplateSpecializationArgsEv
+-Wl,--export=_ZNK5clang12FunctionDecl31getTemplateInstantiationPatternEb
+-Wl,--export=_ZNK5clang17ClassTemplateDecl18getSpecializationsEv
+-Wl,--export=_ZNK5clang29VarTemplateSpecializationDecl22getSpecializedTemplateEv
+-Wl,--export=_ZNK5clang4Decl13getASTContextEv
+-Wl,--export=_ZNK5clang4Decl15hasDefiningAttrEv
+-Wl,--export=_ZNK5clang4Decl8getAttrsEv
+-Wl,--export=_ZN4llvm3sys2fs6accessERKNS_5TwineENS1_10AccessModeE