[PATCH] D107559: [clang] Fix libclang linking on Solaris

[Fangrui Song via Phabricator via cfe-commits]

MaskRay added inline comments.


================
Comment at: clang/tools/libclang/libclang.map:1
-/* If you add a symbol to this file, make sure to add it with the correct
- * version.  For example, if the LLVM main branch is LLVM 14.0.0, add new
- * symbols with the version LLVM_14.
- * On platforms where versions scripts are not used, this file will be used to
- * generate a list of exports for libclang.so
- */
-
+# If you add a symbol to this file, make sure to add it with the correct
+# version.  For example, if the LLVM main branch is LLVM 14.0.0, add new
----------------
MaskRay wrote:
> ro wrote:
> > MaskRay wrote:
> > > GNU ld doesn't support `#` comments. ld.lld supports it.
> > Drats, I'd have sworn they supported `#' comments, too,
> > especially since the version script syntax originated with
> > Sun and was later adopted by GNU `ld`.
> Confirmed that GNU ld and gold support `#` in version scripts, but not in linker scripts.
> 
> ld.lld supports both.
> 
> Using `#` is fine to me if Solaris ld doesn't support `/*` in GNU version scripts.
Correction: only GNU ld doesn't support `#` in linker scripts.

ld.lld and gold support `#` everywhere.

Filed a GNU ld feature request https://sourceware.org/bugzilla/show_bug.cgi?id=28198


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107559