[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