From 5d9e0b41e5a5d30cfe1dec9894c5063938f324cd Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ulysse=20G=C3=A9rard?= Date: Tue, 28 Feb 2023 14:08:04 +0000 Subject: [PATCH] Remove debug or stale comments --- src/dune_rules/uideps.ml | 4 ---- 1 file changed, 4 deletions(-) diff --git a/src/dune_rules/uideps.ml b/src/dune_rules/uideps.ml index 99176136302..e54017e4ab5 100644 --- a/src/dune_rules/uideps.ml +++ b/src/dune_rules/uideps.ml @@ -59,7 +59,6 @@ let cctx_rules cctx = CC.context cctx |> Context.name |> Context_name.build_dir |> Path.build in let* build_path = build_path cctx in - (* let all_cmts = List.map ~f:snd modules_with_cmts in *) let intermediate_targets, intermediates = List.fold_map modules_with_cmts ~init:[] ~f:(fun targets (_module_, for_cmt) -> @@ -73,9 +72,6 @@ let cctx_rules cctx = ; Target fn ; Dep for_cmt ; As (List.map ~f:Path.to_absolute_filename build_path) - (* TODO ulysse: these deps are incorrect. Fixing it might improve - performance by increasing parallelism. *) - (* ; Hidden_deps (Dep.Set.of_files all_cmts) *) ] in (Path.build fn :: targets, action))