Skip to content

Commit

Permalink
[hxb] only enably timers with -D hxb-times
Browse files Browse the repository at this point in the history
  • Loading branch information
kLabz committed May 1, 2024
1 parent 6b3d9e9 commit 3f64224
Show file tree
Hide file tree
Showing 7 changed files with 13 additions and 7 deletions.
5 changes: 5 additions & 0 deletions src-json/define.json
Original file line number Diff line number Diff line change
Expand Up @@ -261,6 +261,11 @@
"platforms": ["hl"],
"params": ["version"]
},
{
"name": "HxbTimes",
"define": "hxb-times",
"doc": "Display hxb timing when used with `--times`."
},
{
"name": "HxcppApiLevel",
"define": "hxcpp-api-level",
Expand Down
2 changes: 1 addition & 1 deletion src/compiler/hxb/hxbLib.ml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ class hxb_library file_path = object(self)
let close = Timer.timer ["hxblib";"read"] in
List.iter (function
| ({ Zip.filename = "StringPool.hxb" | "StringPool.macro.hxb" as filename} as entry) ->
let reader = new HxbReader.hxb_reader (["hxb";"internal"],"StringPool") (HxbReader.create_hxb_reader_stats()) None in
let reader = new HxbReader.hxb_reader (["hxb";"internal"],"StringPool") (HxbReader.create_hxb_reader_stats()) None true (* TODO: -D hxb-times *) in
let zip = Lazy.force zip in
let data = Bytes.unsafe_of_string (Zip.read_entry zip entry) in
ignore(reader#read (new HxbReaderApi.hxb_reader_api_null) data STR);
Expand Down
3 changes: 2 additions & 1 deletion src/compiler/hxb/hxbReader.ml
Original file line number Diff line number Diff line change
Expand Up @@ -149,6 +149,7 @@ class hxb_reader
(mpath : path)
(stats : hxb_reader_stats)
(string_pool : string array option)
(timers_enabled : bool)
= object(self)
val mutable api = Obj.magic ""
val mutable current_module = null_module
Expand Down Expand Up @@ -2000,7 +2001,7 @@ class hxb_reader
method private read_chunk_data kind =
let path = String.concat "_" (ExtLib.String.nsplit (s_type_path mpath) ".") in
let id = ["hxb";"read";string_of_chunk_kind kind;path] in
let close = Timer.timer id in
let close = if timers_enabled then Timer.timer id else fun() -> () in
try
self#read_chunk_data' kind
with Invalid_argument msg -> begin
Expand Down
4 changes: 2 additions & 2 deletions src/compiler/server.ml
Original file line number Diff line number Diff line change
Expand Up @@ -417,7 +417,7 @@ class hxb_reader_api_server
| GoodModule m ->
m
| BinaryModule mc ->
let reader = new HxbReader.hxb_reader path com.hxb_reader_stats (Some cc#get_string_pool_arr) in
let reader = new HxbReader.hxb_reader path com.hxb_reader_stats (Some cc#get_string_pool_arr) (Common.defined com Define.HxbTimes) in
let f_next chunks until =
let t_hxb = Timer.timer ["server";"module cache";"hxb read"] in
let r = reader#read_chunks_until (self :> HxbReaderApi.hxb_reader_api) chunks until in
Expand Down Expand Up @@ -567,7 +567,7 @@ and type_module sctx com delay mpath p =
checking dependencies. This means that the actual decoding never has any reason to fail. *)
begin match check_module sctx mpath mc.mc_extra p with
| None ->
let reader = new HxbReader.hxb_reader mpath com.hxb_reader_stats (Some cc#get_string_pool_arr) in
let reader = new HxbReader.hxb_reader mpath com.hxb_reader_stats (Some cc#get_string_pool_arr) (Common.defined com Define.HxbTimes) in
let api = match com.hxb_reader_api with
| Some api ->
api
Expand Down
2 changes: 1 addition & 1 deletion src/context/display/displayJson.ml
Original file line number Diff line number Diff line change
Expand Up @@ -139,7 +139,7 @@ class hxb_reader_api_com
cc#find_module m_path
with Not_found ->
let mc = cc#get_hxb_module m_path in
let reader = new HxbReader.hxb_reader mc.mc_path com.hxb_reader_stats (Some cc#get_string_pool_arr) in
let reader = new HxbReader.hxb_reader mc.mc_path com.hxb_reader_stats (Some cc#get_string_pool_arr) (Common.defined com Define.HxbTimes) in
fst (reader#read_chunks_until (self :> HxbReaderApi.hxb_reader_api) mc.mc_chunks (if headers_only then MTF else EOM))

method basic_types =
Expand Down
2 changes: 1 addition & 1 deletion src/context/display/displayTexpr.ml
Original file line number Diff line number Diff line change
Expand Up @@ -177,7 +177,7 @@ let check_display_file ctx cs =
| NoModule | BadModule _ -> raise Not_found
| BinaryModule mc ->
let api = (new TypeloadModule.hxb_reader_api_typeload ctx.com ctx.g TypeloadModule.load_module' p :> HxbReaderApi.hxb_reader_api) in
let reader = new HxbReader.hxb_reader path ctx.com.hxb_reader_stats (Some cc#get_string_pool_arr) in
let reader = new HxbReader.hxb_reader path ctx.com.hxb_reader_stats (Some cc#get_string_pool_arr) (Common.defined ctx.com Define.HxbTimes) in
let m = reader#read_chunks api mc.mc_chunks in
m
| GoodModule m ->
Expand Down
2 changes: 1 addition & 1 deletion src/typing/typeloadModule.ml
Original file line number Diff line number Diff line change
Expand Up @@ -779,7 +779,7 @@ let rec load_hxb_module com g path p =
let read file bytes string_pool =
try
let api = (new hxb_reader_api_typeload com g load_module' p :> HxbReaderApi.hxb_reader_api) in
let reader = new HxbReader.hxb_reader path com.hxb_reader_stats string_pool in
let reader = new HxbReader.hxb_reader path com.hxb_reader_stats string_pool (Common.defined com Define.HxbTimes) in
let read = reader#read api bytes in
let m = read EOT in
delay g PConnectField (fun () ->
Expand Down

0 comments on commit 3f64224

Please sign in to comment.