[PATCH] D90831: DebugInfo support for OCaml bindings
Josh Berdine via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 17 04:18:13 PDT 2021
jberdine added a comment.
I see, it seems some of the builds are still using a version of OCaml older than 4.06 (from 11-2017). The difference between using `caml_alloc_initialized_string` and the code with `memcpy` is about `const char *` versus `char *`. If OCaml is configured with immutable strings (the default starting with 4.10), then `String_val` returns a `const char *`. I guess that supporting such old versions will necessitate compilation warnings about `const`.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D90831/new/
https://reviews.llvm.org/D90831
More information about the llvm-commits
mailing list