aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | Merge #2734bors[bot]2020-01-032-1/+18
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2734: Fix #2705 r=flodiebold a=flodiebold The `-` turned into a `+` during a refactoring. The original issue was caused by `Read` resolving wrongly to a trait without type parameters instead of a struct with one parameter; this only fixes the crash, not the wrong resolution. Co-authored-by: Florian Diebold <[email protected]>
| * | | Fix #2705Florian Diebold2020-01-032-1/+18
|/ / / | | | | | | | | | | | | | | | | | | | | | The `-` turned into a `+` during a refactoring. The original issue was caused by `Read` resolving wrongly to a trait without type parameters instead of a struct with one parameter; this only fixes the crash, not the wrong resolution.
* | | Merge #2731bors[bot]2020-01-034-9/+18
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2731: Split `crate_def_map` into two methods r=matklad a=michalt This change: - introduces `compute_crate_def_map` query and renames `CrateDefMap::crate_def_map_query` for consistency, - annotates `crate_def_map` as `salsa::transparent` and adds a top-level `crate_def_map` wrapper function around that starts the profiler and immediately calls into `compute_crate_def_map` query. This allows us to better understand where we spent the time, in particular, how much is spent in the recomputaiton and how much in salsa. Example output (where we don't actually re-compute anything, but the query still takes a non-trivial amount of time): ``` 211ms - handle_inlay_hints 150ms - get_inlay_hints 150ms - SourceAnalyzer::new 65ms - def_with_body_from_child_node 65ms - analyze_container 65ms - analyze_container 65ms - Module::from_definition 65ms - Module::from_file 65ms - crate_def_map 1ms - parse_macro_query (6 calls) 0ms - raw_items_query (1 calls) 64ms - ??? ``` Signed-off-by: Michal Terepeta <[email protected]> Co-authored-by: Michal Terepeta <[email protected]>
| * | | Add `_query` suffix to `compute_crate_def_map` for consistencyMichal Terepeta2020-01-032-2/+2
| | | | | | | | | | | | | | | | Signed-off-by: Michal Terepeta <[email protected]>
| * | | Split `crate_def_map` into two methodsMichal Terepeta2020-01-034-9/+18
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change: - introduces `compute_crate_def_map` query and renames `CrateDefMap::crate_def_map_query` for consistency, - annotates `crate_def_map` as `salsa::transparent` and adds a top-level `crate_def_map` wrapper function around that starts the profiler and immediately calls into `compute_crate_def_map` query. This allows us to better understand where we spent the time, in particular, how much is spent in the recomputaiton and how much in salsa. Example output (where we don't actually re-compute anything, but the query still takes a non-trivial amount of time): ``` 211ms - handle_inlay_hints 150ms - get_inlay_hints 150ms - SourceAnalyzer::new 65ms - def_with_body_from_child_node 65ms - analyze_container 65ms - analyze_container 65ms - Module::from_definition 65ms - Module::from_file 65ms - crate_def_map 1ms - parse_macro_query (6 calls) 0ms - raw_items_query (1 calls) 64ms - ??? ``` Signed-off-by: Michal Terepeta <[email protected]>
* | | Merge #2730bors[bot]2020-01-031-0/+3
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2730: Add `profile` calls to {Module,Function}::diagnostics r=matklad a=michalt With this change the output `ra_prof` gives a better indication where the time is spent. Example output: ``` 213ms - publish_diagnostics 213ms - diagnostics 70ms - Module::from_definition 70ms - Module::from_file 132ms - Module::diagnostics 78ms - Function::diagnostics 0ms - body_with_source_map_query (1 calls) 2ms - trait_solve_query (1 calls) 76ms - ??? 15ms - Function::diagnostics 0ms - body_with_source_map_query (1 calls) 15ms - trait_solve_query (5 calls) 38ms - Function::diagnostics (51 calls) 8ms - parse_query (1 calls) ``` Signed-off-by: Michal Terepeta <[email protected]> Co-authored-by: Michal Terepeta <[email protected]>
| * | | Add `profile` calls to {Module,Function}::diagnosticsMichal Terepeta2020-01-031-0/+3
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With this change the output `ra_prof` gives a better indication where the time is spent. Example output: ``` 213ms - publish_diagnostics 213ms - diagnostics 70ms - Module::from_definition 70ms - Module::from_file 132ms - Module::diagnostics 78ms - Function::diagnostics 0ms - body_with_source_map_query (1 calls) 2ms - trait_solve_query (1 calls) 76ms - ??? 15ms - Function::diagnostics 0ms - body_with_source_map_query (1 calls) 15ms - trait_solve_query (5 calls) 38ms - Function::diagnostics (51 calls) 8ms - parse_query (1 calls) ``` Signed-off-by: Michal Terepeta <[email protected]>
* | | Merge #2729bors[bot]2020-01-031-1/+0
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | 2729: Remove a duplicate line in `collect_defs` r=matklad a=michalt Co-authored-by: Michal Terepeta <[email protected]>
| * | Remove a duplicate line in `collect_defs`Michal Terepeta2020-01-031-1/+0
|/ / | | | | | | Signed-off-by: Michal Terepeta <[email protected]>
* | Merge #2723bors[bot]2020-01-014-25/+26
|\ \ | | | | | | | | | | | | | | | | | | | | | 2723: Update dependencies r=kjeremy a=kjeremy Co-authored-by: Jeremy Kolb <[email protected]>
| * | Update dependenciesJeremy Kolb2020-01-014-25/+26
|/ /
* | Merge #2722bors[bot]2020-01-011-0/+7
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2722: Fix ra_cargo_watch test on windows. r=matklad a=kiljacken Due to difference in path parsing on platforms, the current input to the snapshot tests in ra_cargo_watch cause a panic when running on Windows. This PR disables the tests when running on Windows until a better solution is found. If somebody running on Windows could test this that would be great, as I'm working in the blind here :) Fixes #2720 (probably). Co-authored-by: Emil Lauridsen <[email protected]>
| * | Disable cargo_watch snapshot tests on windowsEmil Lauridsen2020-01-011-0/+7
|/ /
* | Merge #2718bors[bot]2020-01-011-1/+2
|\ \ | | | | | | | | | | | | | | | | | | | | | 2718: Allow disabling inlay hints r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | Allow disabling inlay hintsAleksey Kladov2020-01-011-1/+2
|/ /
* | Merge #2715bors[bot]2019-12-312-17/+12
|\ \ | | | | | | | | | | | | | | | | | | | | | 2715: Fixes to progress display r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | Fixes to progress displayAleksey Kladov2019-12-312-17/+12
| |/
* | Merge pull request #2672 from Speedy37/masterAleksey Kladov2019-12-312-37/+83
|\ \ | |/ |/| fix #2520: change expand_repeat loop stop condition
| * Add details about test_repeat_bad_var current stateVincent Rouillé2019-12-301-0/+2
| |
| * Details about macro NestingState hit and at_end fieldsVincent Rouillé2019-12-301-7/+10
| |
| * fix #2520: change expand_repeat loop stop conditionVincent Rouillé2019-12-282-36/+77
| |
* | Merge #2711bors[bot]2019-12-3114-38/+160
|\ \ | | | | | | | | | | | | | | | | | | | | | 2711: Add semicolons r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | Add semicolonsAleksey Kladov2019-12-3114-38/+160
| | |
* | | Merge #2710bors[bot]2019-12-315-21/+24
|\| | | | | | | | | | | | | | | | | | | | | | | 2710: Fix NPEs r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | Fix NPEsAleksey Kladov2019-12-315-21/+24
| | |
* | | Merge #2709bors[bot]2019-12-3113-223/+235
|\| | | | | | | | | | | | | | | | | | | | | | | 2709: Work around synchrnonisation issue r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | Work around synchrnonisation issueAleksey Kladov2019-12-311-6/+11
| | |
| * | Refactor server lifecycleAleksey Kladov2019-12-3112-201/+218
| | |
| * | Refactor configAleksey Kladov2019-12-312-7/+6
| | |
| * | Drop needless pubsAleksey Kladov2019-12-311-14/+14
| | |
| * | Move config to CtxAleksey Kladov2019-12-313-7/+5
| | |
| * | Refactor status activationAleksey Kladov2019-12-315-45/+38
|/ /
* | Merge #2706bors[bot]2019-12-312-2/+2
|\ \ | | | | | | | | | | | | | | | | | | | | | 2706: Fix color name r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | Fix color nameAleksey Kladov2019-12-312-2/+2
|/ /
* | Merge #2704bors[bot]2019-12-311-10/+3
|\ \ | | | | | | | | | | | | | | | | | | | | | 2704: Add example of color customization r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | Add example of color customizationAleksey Kladov2019-12-311-10/+3
| | |
* | | Merge #2703bors[bot]2019-12-312-289/+2
|\| | | | | | | | | | | | | | | | | | | | | | | 2703: Drop obsolete color configurations r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | Drop obsolete color configurationsAleksey Kladov2019-12-312-289/+2
|/ /
* | Merge #2702bors[bot]2019-12-318-271/+187
|\ \ | | | | | | | | | | | | | | | | | | | | | 2702: Drop support for legacy colorization r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | Drop support for legacy colorizationAleksey Kladov2019-12-318-271/+187
| | |
| * | Rename fileAleksey Kladov2019-12-313-2/+2
|/ /
* | Merge #2701bors[bot]2019-12-312-4/+4
|\ \ | | | | | | | | | | | | | | | | | | | | | 2701: :arrow_up: lsp-server r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | :arrow_up: lsp-serverAleksey Kladov2019-12-311-3/+3
| | |
| * | Fix #2700Aleksey Kladov2019-12-311-1/+1
|/ /
* | Merge #2699bors[bot]2019-12-315-221/+182
|\ \ | | | | | | | | | | | | | | | | | | | | | 2699: Switch impure functional style to pure imperative r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | Switch impure functional style to pure imperativeAleksey Kladov2019-12-314-95/+67
| | |
| * | Reduce scopeAleksey Kladov2019-12-311-8/+10
| | |
| * | Use optional value shortcutAleksey Kladov2019-12-311-3/+3
| | |
| * | Remove trivial helpersAleksey Kladov2019-12-311-15/+3
| | |
| * | Rearrange codeAleksey Kladov2019-12-312-107/+106
|/ /