tstellar added inline comments.

================
Comment at: clang/tools/libclang/CMakeLists.txt:82
+
+if (LLVM_EXPORTED_SYMBOL_FILE)
+  add_custom_command(OUTPUT ${LLVM_EXPORTED_SYMBOL_FILE}
----------------
MaskRay wrote:
> What does this do?
> 
> A hard-coded list cannot catch up with the real dynamic symbol list. We'll 
> have to parse llvm-readelf --dyn-syms output...
Are you saying we should generate the version script using llvm-readelf?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D105527/new/

https://reviews.llvm.org/D105527

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to