Add a comment about cache behavior in case of PP

ocaml · May 11, 2023 · cfb2fed · cfb2fed
1 parent a7c61a6
commit cfb2fed
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/kernel/mpipeline.ml b/src/kernel/mpipeline.ml
@@ -245,7 +245,10 @@ let process
             let cache_disabling =
               match (config.merlin.use_ppx_cache, pp_result) with
               | false, _ -> Some "configuration"
-              | true, Some _ -> Some "source preprocessor usage"
+              | true, Some _ ->
+                (* The cache could be refined in the future to also act on the PP phase.
+                   For now, let's disable the whole cache when there's a PP. *)
+                Some "source preprocessor usage"
               | true, None -> None
             in
             Reader_with_cache.apply ~cache_disabling ~force_invalidation:false