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

Author: mcbarton

.github/workflows/MacOS.yml

@@ -389,8 +389,7 @@ jobs:
                 -DLLVM_ENABLE_WERROR=On                      \
                 ../
         fi
-        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
+        cmake --build . --target check-cppinterop --parallel ${{ env.ncpus }}
         cd ..
         # We need CB_PYTHON_DIR later
         echo "CB_PYTHON_DIR=$CB_PYTHON_DIR" >> $GITHUB_ENV

lib/Interpreter/CMakeLists.txt

@@ -115,11 +115,9 @@ endif()
   )
 
 if(EMSCRIPTEN)
-  # Read the undefined symbols from the text file
-  file(READ "${CMAKE_SOURCE_DIR}/lib/Interpreter/undefined_symbols.txt" SYMBOLS_LIST)
+  # Specify the path to your linker script
+  set(LINKER_SCRIPT_PATH "${CMAKE_SOURCE_DIR}/lib/Interpreter/exports.ld")
 
-  # 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
@@ -128,10 +126,14 @@ 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 ${SYMBOLS_LIST}"
+    LINK_FLAGS "-s WASM_BIGINT  -s SIDE_MODULE=1"
     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,65 @@
+EXPORTS
+{
+    _ZN4llvm3sys2fs17getMainExecutableEPKcPv;
+    _ZN4llvm3sys4path11parent_pathENS_9StringRefENS1_5StyleE;
+    _ZN3Cpp11GetOperatorEPvNS_8OperatorERNSt3__26vectorIS0_NS2_9allocatorIS0_EEEENS_13OperatorArityE;
+    _ZN4llvm11raw_ostream14flush_nonemptyEv;
+    _ZN4llvm11raw_ostream16SetBufferAndModeEPcmNS0_10BufferKindE;
+    _ZN4llvm11raw_ostream5writeEPKcm;
+    _ZN4llvm11raw_ostreamD2Ev;
+    _ZN4llvm13StringMapImpl11RehashTableEj;
+    _ZN4llvm13StringMapImpl15LookupBucketForENS_9StringRefEj;
+    _ZN4llvm13StringMapImpl4hashENS_9StringRefE;
+    _ZN4llvm15SmallVectorBaseIjE8grow_podEPvmm;
+    _ZN4llvm15allocate_bufferEmm;
+    _ZN4llvm21logAllUnhandledErrorsENS_5ErrorERNS_11raw_ostreamENS_5TwineE;
+    _ZN4llvm25llvm_unreachable_internalEPKcS1_j;
+    _ZN4llvm3sys17RunningOnValgrindEv;
+    _ZN4llvm3sys4path6appendERNS_15SmallVectorImplIcEERKNS_5TwineES7_S7_S7_;
+    _ZN4llvm4dbgsEv;
+    _ZN4llvm4errsEv;
+    _ZN4llvm5APIntC1EjNS_8ArrayRefIyEE;
+    _ZN5clang10ASTContext19createMangleContextEPKNS_10TargetInfoE;
+    _ZN5clang11DeclContext7classofEPKNS_4DeclE;
+    _ZN5clang11Interpreter19getCompilerInstanceEv;
+    _ZN5clang11Interpreter5ParseEN4llvm9StringRefE;
+    _ZN5clang11Interpreter6createENSt3__210unique_ptrINS_16CompilerInstanceENS1_14default_deleteIS3_EEEE;
+    _ZN5clang11Interpreter7ExecuteERNS_22PartialTranslationUnitE;
+    _ZNK5clang4Sema15getStdNamespaceEv;
+    _ZNK5clang4Type27getUnqualifiedDesugaredTypeEv;
+    _ZNK5clang7TagType7getDeclEv;
+    _ZNK5clang7VarDecl28isThisDeclarationADefinitionERNS_10ASTContextE;
+    _ZTVN4llvm18raw_string_ostreamE;
+    clang_Interpreter_dispose;
+    clang_Interpreter_getClangInterpreter;
+    clang_Interpreter_takeInterpreterAsPtr;
+    clang_createInterpreterFromRawPtr;
+    clang_getComplexType;
+    clang_getTypeAsString;
+    clang_instantiateTemplate;
+    _ZN5clang13MangleContext10mangleNameENS_10GlobalDeclERN4llvm11raw_ostreamE;
+    _ZN5clang13MangleContext20shouldMangleDeclNameEPKNS_9NamedDeclE;
+    _ZN5clang26IncrementalCompilerBuilder9CreateCppEv;
+    _ZN5clang4Decl17castToDeclContextEPKS0_;
+    _ZNK3Cpp7JitCall17AreArgumentsValidEPvNS0_7ArgListES1_;
+    _ZNK3Cpp7JitCall17ReportInvokeStartEPvNS0_7ArgListES1_;
+    _ZNK3Cpp7JitCall17ReportInvokeStartEPvmi;
+    _ZNK4llvm5APInt13compareSignedERKS0_;
+    _ZNK4llvm5APInt4sextEj;
+    _ZNK4llvm5APInt4zextEj;
+    _ZNK4llvm5APInt7compareERKS0_;
+    _ZNK4llvm5Error19fatalUncheckedErrorEv;
+    _ZNK5clang10ASTContext14getComplexTypeENS_8QualTypeE;
+    _ZNK5clang10ASTContext19getTypeDeclTypeSlowEPKNS_8TypeDeclE;
+    _ZNK5clang10RecordDecl19isInjectedClassNameEv;
+    _ZNK5clang11DeclContext11decls_beginEv;
+    _ZNK5clang11DeclContext6lookupENS_15DeclarationNameE;
+    _ZNK5clang12FunctionDecl29getTemplateSpecializationArgsEv;
+    _ZNK5clang12FunctionDecl31getTemplateInstantiationPatternEb;
+    _ZNK5clang17ClassTemplateDecl18getSpecializationsEv;
+    _ZNK5clang29VarTemplateSpecializationDecl22getSpecializedTemplateEv;
+    _ZNK5clang4Decl13getASTContextEv;
+    _ZNK5clang4Decl15hasDefiningAttrEv;
+    _ZNK5clang4Decl8getAttrsEv;
+    _ZN4llvm3sys2fs6accessERKNS_5TwineENS1_10AccessModeE;
+}