[llvm] [OCaml] Use DataLayout instead of string in (set_)data_layout (PR #228445)

[Ramkumar Ramachandra via llvm-commits]

================
@@ -328,14 +328,14 @@ value llvm_set_target_triple(value Trip, value M) {
   return Val_unit;
 }
 
-/* llmodule -> string */
+/* llmodule -> DataLayout.t */
 value llvm_data_layout(value M) {
-  return caml_copy_string(LLVMGetDataLayout(Module_val(M)));
+  return llvm_alloc_data_layout(LLVMGetModuleDataLayout(Module_val(M)));
 }
 
-/* string -> llmodule -> unit */
+/* DataLayout.t -> llmodule -> unit */
 value llvm_set_data_layout(value Layout, value M) {
-  LLVMSetDataLayout(Module_val(M), String_val(Layout));
+  LLVMSetModuleDataLayout(Module_val(M), DataLayout_val(Layout));
----------------
artagnon wrote:

Can you kindly explain this change as well? Has the function name changed from LLVMSetDataLayout -> LLVMSetModuleDataLayout in some invisible manner?

https://github.com/llvm/llvm-project/pull/228445