YJIT: Make timestamps the first element in the YJIT compilation log t…

…uple.
Shopify · Oct 7, 2024 · fb9557d · fb9557d
1 parent c35877a
commit fb9557d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/yjit.rb b/yjit.rb
@@ -191,8 +191,8 @@ def self.stats_string
   def self.compilation_log
     return nil unless compilation_log_enabled?
 
-    Primitive.rb_yjit_get_compilation_log.map do |path, timestamp|
-      [path, Time.at(timestamp)]
+    Primitive.rb_yjit_get_compilation_log.map do |timestamp, path|
+      [Time.at(timestamp), path]
     end
   end
 

diff --git a/yjit/src/compilation_log.rs b/yjit/src/compilation_log.rs
@@ -230,8 +230,8 @@ fn rb_yjit_get_compilation_log_array() -> VALUE {
     for entry in log.iter() {
         unsafe {
             let entry_array = rb_ary_new_capa(2);
-            rb_ary_push(entry_array, entry.payload.to_string().into());
             rb_ary_push(entry_array, rb_float_new(entry.timestamp));
+            rb_ary_push(entry_array, entry.payload.to_string().into());
             rb_ary_push(array, entry_array);
         }
     }