[llvm] 20799fd - [OCaml][test] Use correct data layout string. (#68781)

via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 11 03:12:09 PDT 2023


Author: Harald van Dijk
Date: 2023-10-11T11:12:04+01:00
New Revision: 20799fd57bcbc46eb9a26cd2307c8fc8b935a424

URL: https://github.com/llvm/llvm-project/commit/20799fd57bcbc46eb9a26cd2307c8fc8b935a424
DIFF: https://github.com/llvm/llvm-project/commit/20799fd57bcbc46eb9a26cd2307c8fc8b935a424.diff

LOG: [OCaml][test] Use correct data layout string. (#68781)

core.ml would previously set a data layout string of "e" and check that
it remained "e". This is fragile when we have the data layout string
auto-upgrade facility, and indeed broke when D86310 upgraded this one.
As the auto-upgrade logic is not what is being tested here, it seems
easier to just use the data layout string that the target expects.

Added: 
    

Modified: 
    llvm/test/Bindings/OCaml/core.ml

Removed: 
    


################################################################################
diff  --git a/llvm/test/Bindings/OCaml/core.ml b/llvm/test/Bindings/OCaml/core.ml
index 3a9c3c84238d702..1f6ece9532f1943 100644
--- a/llvm/test/Bindings/OCaml/core.ml
+++ b/llvm/test/Bindings/OCaml/core.ml
@@ -71,11 +71,11 @@ let test_target () =
   end;
 
   begin group "layout";
-    let layout = "e" in
+    let layout = "e-m:o-p:32:32-p270:32:32-p271:32:32-p272:64:64-i128:128-f64:32:64-f80:128-n8:16:32-S128" in
     set_data_layout layout m;
     insist (layout = data_layout m)
   end
-  (* CHECK: target datalayout = "e"
+  (* CHECK: target datalayout = "e-m:o-p:32:32-p270:32:32-p271:32:32-p272:64:64-i128:128-f64:32:64-f80:128-n8:16:32-S128"
    * CHECK: target triple = "i686-apple-darwin8"
    *)
 


        


More information about the llvm-commits mailing list