[ConfigImpact] Revise checking of having known expensive calls

Summary: It missed `unchecked_callees_cond`, which introduced FNs.

Reviewed By: ezgicicek

Differential Revision: D28088784

fbshipit-source-id: bfd5d92d0
master
Sungkeun Cho 4 years ago committed by Facebook GitHub Bot
parent 46c0c0297b
commit 08ea9eaee8

@ -138,6 +138,10 @@ module UncheckedCalleesCond = struct
map (UncheckedCallees.replace_location_by_call location) fields_map map (UncheckedCallees.replace_location_by_call location) fields_map
let has_known_expensive_callee fields_map =
exists (fun _ callees -> UncheckedCallees.has_known_expensive_callee callees) fields_map
let filter_known_expensive fields_map = let filter_known_expensive fields_map =
filter_map filter_map
(fun _ callees -> (fun _ callees ->
@ -225,8 +229,9 @@ module Summary = struct
{x with unchecked_callees; unchecked_callees_cond= UncheckedCalleesCond.bottom} {x with unchecked_callees; unchecked_callees_cond= UncheckedCalleesCond.bottom}
let has_known_expensive_callee {unchecked_callees} = let has_known_expensive_callee {unchecked_callees; unchecked_callees_cond} =
UncheckedCallees.has_known_expensive_callee unchecked_callees UncheckedCallees.has_known_expensive_callee unchecked_callees
|| UncheckedCalleesCond.has_known_expensive_callee unchecked_callees_cond
end end
module Dom = struct module Dom = struct

Loading…
Cancel
Save