From a2ba4016a64f84193261db9a52196adc39cb5767 Mon Sep 17 00:00:00 2001
From: Pierre Roux
Date: Thu, 12 Sep 2019 19:02:18 +0200
Subject: Reimplement is_float

is_float was relying on Obj.tag triggering a check that we are in the
OCaml heap which is expensive. On extreme examples, this can lead to a
global 2x speedup.

Thanks to Maxime Dénès and Jacques-Henri Jourdan for their help in
diagnosing this.
---
 kernel/nativevalues.ml | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

(limited to 'kernel/nativevalues.ml')

diff --git a/kernel/nativevalues.ml b/kernel/nativevalues.ml
index 1d4fb5d39c..6b9d49052d 100644
--- a/kernel/nativevalues.ml
+++ b/kernel/nativevalues.ml
@@ -517,10 +517,8 @@ let print x =
 
 (** Support for machine floating point values *)
 
-let is_float (x:t) =
-  let o = Obj.repr x in
-  Int.equal (Obj.tag o) Obj.double_tag
-[@@ocaml.inline always]
+external is_float : t -> bool = "coq_is_double"
+[@@noalloc]
 
 let to_float (x:t) = (Obj.magic x : Float64.t)
 [@@ocaml.inline always]
-- 
cgit v1.2.3