23 Commits (757460ade7ecad545b0ad1647eee4bc3f5254401)

Author	SHA1	Message	Date
Mehdi Bouaziz	4d5f8e6fea	Small refactorings: Hoisting ... Summary: - no `Ondemand` -> `get_callee_cost_summary_and_formals` - explicit dependency on `Config` -> pass `should_report_invariant` around - split reporting part of `checker` - mli Reviewed By: ezgicicek Differential Revision: D14258480 fbshipit-source-id: 08589d12e	6 years ago
Mehdi Bouaziz	21c9227529	Small refactorings: LoopInvariant ... Summary: - hide `Ondemand` -> pass `get_callee_purity` around - mli Reviewed By: ezgicicek Differential Revision: D14258400 fbshipit-source-id: 466e5dbaa	6 years ago
Mehdi Bouaziz	5a5a865bf4	Small refactorings: ReachingDefs ... Summary: Adds an mli and don't expose the internal Analyzer. Reviewed By: ezgicicek Differential Revision: D14258324 fbshipit-source-id: ecc433226	6 years ago
Mehdi Bouaziz	b212f1ce6c	[hoisting] Fix reporting logic ... Summary: We don't want to use Cost analysis results when `Config.hoisting_report_only_expensive` is false Reviewed By: ezgicicek Differential Revision: D14124555 fbshipit-source-id: e809bb80a	6 years ago
Ezgi Çiçek	cd20abfc88	[cost] Add trace to symbols in polynomial bounds ... Summary: Record where each symbol in a polynomial is coming from: either a loop, function call or a modeled call. Reviewed By: mbouaziz Differential Revision: D14047420 fbshipit-source-id: 56d0bd926	6 years ago
Mehdi Bouaziz	3139ee5d46	[cost] Instantiate costs using formals not pname ... Summary: Simplifies `instantiate_cost`, call `Ondemand` once per call site. Depends on D14028249 Reviewed By: ddino Differential Revision: D14028281 fbshipit-source-id: 865151fee	6 years ago
Dino Distefano	b52f2eb4be	Simulate record domain in cost analysis ... Reviewed By: mbouaziz Differential Revision: D13971520 fbshipit-source-id: a43ff1329	6 years ago
Ezgi Çiçek	50b1533921	[loop-invariance] Make InvariantModels based on ProcName ... Reviewed By: mbouaziz Differential Revision: D13971492 fbshipit-source-id: c47d6b858	6 years ago
Dino Distefano	e6d2872a4e	Using record data structure for the cost domain ... Reviewed By: mbouaziz Differential Revision: D13897572 fbshipit-source-id: 4148b18b4	6 years ago
Mehdi Bouaziz	3fabbfdcad	[inferbo] Split Analysis and Checker ... Reviewed By: skcho Differential Revision: D13852741 fbshipit-source-id: f0da54257	6 years ago
Mehdi Bouaziz	8a3592c34e	[inferbo] Uncouple numerical analysis and numerical checks ... Summary: This will allow to get the numerical results for Cost, Hoisting, Purity without the Inferbo issues. For now, I still forced Inferbo issues for Cost and Purity to avoid lots of changes in tests, that will go away soon. Reviewed By: ezgicicek, skcho Differential Revision: D13826741 fbshipit-source-id: 796d1a50d	6 years ago
Ezgi Çiçek	61b51b09db	[hoisting] Rename hoisting message ... Reviewed By: ngorogiannis Differential Revision: D13415995 fbshipit-source-id: bbf1ee855	6 years ago
Mehdi Bouaziz	b0b96b453b	[cost] Do not need to load inferbo summary at calls ... Reviewed By: skcho Differential Revision: D13190966 fbshipit-source-id: 4e745dac3	6 years ago
Ezgi Çiçek	305b2a74b8	[hoisting] Add new issue type for only invariant-modeled calls ... Reviewed By: ngorogiannis Differential Revision: D13194812 fbshipit-source-id: 0c020789d	6 years ago
Mehdi Bouaziz	8292323307	[inferbo/cost] Move Polynomials out of Itv.ml ... Reviewed By: skcho Differential Revision: D13173440 fbshipit-source-id: d1e699ee6	6 years ago
Sungkeun Cho	c8a17b9d0e	[inferbo] Pass integer type widths to eval for cast ... Summary: In this diff, it passes the parameter of integer type widths to evaluation functions. The parameter which will be used for casting in the following diff. Reviewed By: mbouaziz Differential Revision: D12920581 fbshipit-source-id: 48bbc802b	6 years ago
Ezgi Çiçek	f3d82a0230	[hoisting] Don't report functions modeled as VariantForHoisting but consider them invariant ... Reviewed By: ddino Differential Revision: D13025250 fbshipit-source-id: ba1e39591	6 years ago
Ezgi Çiçek	99c2a6da8d	[hoisting] Hoist only expensive pure functions ... Reviewed By: mbouaziz Differential Revision: D10236706 fbshipit-source-id: c51a9ff0c	6 years ago
Mehdi Bouaziz	2be4710811	[AI] Enable debugging only based on --write-html ... Reviewed By: ezgicicek Differential Revision: D10238681 fbshipit-source-id: 08eeedd26	7 years ago
Mehdi Bouaziz	ddbb7e05d3	Reporting cleanup 22: log_error/warning -> use IssueType rather than exception ... Summary: Callsites of `Reporting.log_error/warning` always use `Exceptions.Checkers`, let's simplify the API. Under the hood it still creates an exception, but this can be cleaned up later. Reviewed By: jeremydubreil Differential Revision: D9799860 fbshipit-source-id: 6492a60b4	7 years ago
Ezgi Çiçek	15bdcca119	[Loop-hoisting] Add a new model for Javax's Provider.get() and make it invariant only ... Reviewed By: mbouaziz, ddino Differential Revision: D9748162 fbshipit-source-id: bedd2300e	7 years ago
Josh Berdine	40ab73037e	[ocamlformat] upgrade to ocamlformat 0.7 ... Reviewed By: mbouaziz Differential Revision: D9496601 fbshipit-source-id: 83c6fd241	7 years ago
Ezgi Çiçek	49e582fa49	[Loop-hoisting] Add a new checker for hoisting invariant function calls ... Reviewed By: mbouaziz Differential Revision: D9460455 fbshipit-source-id: 7488c1a69	7 years ago