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
----------------
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.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107559

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

Reply via email to