aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #2762 from edwin0cheng/fix-2759Aleksey Kladov2020-01-081-11/+25
|\ | | | | Reject tuple index for missing fields assist
| * Proper emit diagnostic without fixEdwin Cheng2020-01-071-19/+25
| |
| * Reject tuple index for missing fields assistEdwin Cheng2020-01-071-0/+8
| |
* | Merge #2763bors[bot]2020-01-082-6/+6
|\ \ | | | | | | | | | | | | | | | | | | | | | 2763: :arrow_up: pico-args r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | :arrow_up: pico-argsAleksey Kladov2020-01-082-6/+6
|/ /
* | Merge pull request #2760 from edwin0cheng/fix-descend-into-macrosAleksey Kladov2020-01-071-2/+3
|\ \ | | | | | | Use src node for analyze source in `descend_into_macros`
| * | Use src node for analyze sourceEdwin Cheng2020-01-071-2/+3
| |/
* | Merge pull request #2761 from matklad/cachingAleksey Kladov2020-01-071-1/+15
|\ \ | | | | | | Try to restore caching on CI
| * | More tenacious CI networkingAleksey Kladov2020-01-071-0/+2
| | |
| * | Try to restore caching on CIAleksey Kladov2020-01-071-1/+13
| | |
* | | Merge #2758bors[bot]2020-01-071-1/+1
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2758: Add "code-oss" Visual Studio Code binary name r=matklad a=leo-lb When Visual Studio Code is manually compiled from the upstream source tree the binary is named "code-oss". Co-authored-by: leo-lb <[email protected]>
| * | | Add "code-oss" Visual Studio Code binary nameleo-lb2020-01-071-1/+1
| | | | | | | | | | | | | | | | When Visual Studio Code is manually compiled from the upstream source tree the binary is named "code-oss".
* | | | Merge pull request #2756 from matklad/retasksAleksey Kladov2020-01-075-44/+71
|\ \ \ \ | |/ / / |/| / / | |/ / :arrow_up: crates
| * | Fix back compatAleksey Kladov2020-01-071-0/+1
| | |
| * | :arrow_up: cratesAleksey Kladov2020-01-071-22/+21
| | |
| * | Share cache cleaning logic between OSesAleksey Kladov2020-01-073-22/+49
| | |
* | | Merge #2755bors[bot]2020-01-076-371/+344
|\| | | | | | | | | | | | | | | | | | | | | | | 2755: Cleanup r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | CleanupAleksey Kladov2020-01-073-7/+7
| | |
| * | Move cmd to a separate dirAleksey Kladov2020-01-072-53/+58
| | |
| * | Refactor xtasksAleksey Kladov2020-01-075-281/+290
| | |
| * | SimplifyAleksey Kladov2020-01-072-45/+4
| |/ | | | | | | help for internal tools does not worth the duplication
* | Merge #2754bors[bot]2020-01-071-1/+4
|\ \ | |/ |/| | | | | | | | | | | 2754: Fix panic in tuple fields r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * Fix panic in tuple fieldsAleksey Kladov2020-01-071-1/+4
|/
* Merge pull request #2725 from kiljacken/actions-matrixAleksey Kladov2020-01-072-3/+35
|\ | | | | Run tests on ubuntu, windows and macos
| * Add missing comma and use double-quotes to bors.tomlEmil Lauridsen2020-01-051-3/+3
| |
| * Rename old toolchains on windows to speedup installationEmil Lauridsen2020-01-031-2/+6
| |
| * Remove rust-docs on Windows before updating to latest stableEmil Lauridsen2020-01-031-0/+4
| |
| * Update bors.tomlEmil Lauridsen2020-01-031-1/+6
| |
| * Blindfolded PowerShell codingEmil Lauridsen2020-01-031-1/+17
| |
| * Run tests on ubuntu, windows and macosEmil Lauridsen2020-01-031-1/+4
| |
* | Merge #2724bors[bot]2020-01-069-21/+123
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2724: Improve const generic parsing r=matklad a=mchesser Add support for generic arguments in: `impl` type parameters: ```rust impl<const N: u32> Bar<N> {} ``` type args: ```rust type A = B<1, { 2 }>; test::<10>(); ``` Co-authored-by: Michael Chesser <[email protected]>
| * | Improve const generics parsingMichael Chesser2020-01-069-21/+123
|/ / | | | | | | | | - Handle const generics type args - Fix issue with const generic as first parameter in trait impl
* | Merge #2753bors[bot]2020-01-061-1/+1
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2753: Fix a problem with `Durability` of libraries r=matklad a=michalt When processing a change with added libraries, we used `Default::default` for `SourceRoot` which sets `is_library` to false. Since we use `is_library` to decide whether to use low or high durability, I believe that this caused us to mark many library dependencies as having low durability and thus increased the size of the graph that salsa needed to verify on every change. Based on my initial tests this speeds up the `CrateDefMapQuery` on rust-analyzer from about ~64ms to ~14ms and reduces the number of validations for the query from over 60k to about 7k. Signed-off-by: Michal Terepeta <[email protected]> Co-authored-by: Michal Terepeta <[email protected]>
| * | Fix a problem with `Durability` of librariesMichal Terepeta2020-01-061-1/+1
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | When processing a change with added libraries, we used `Default::default` for `SourceRoot` which sets `is_library` to false. Since we use `is_library` to decide whether to use low or high durability, I believe that this caused us to mark many library dependencies as having low durability and thus increased the size of the graph that salsa needed to verify on every change. Based on my initial tests this speeds up the `CrateDefMapQuery` on rust-analyzer from about ~64ms to ~14ms and reduces the number of validations for the query from over 60k to about 7k. Signed-off-by: Michal Terepeta <[email protected]>
* | Merge #2750bors[bot]2020-01-061-0/+4
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | 2750: add note in vscode instructions for vim plugin users r=matklad a=guswynn Fixes #2746 As the issue mentions, this is already mentioned below in the setting docs. However, it ended up taking me a long time to figure this out, so a note in the instructions for Vscode specifically is helpful Co-authored-by: Gus Wynn <[email protected]>
| * | add note in vscode instructions for vim plugin usersGus Wynn2020-01-051-0/+4
| | |
* | | Merge #2714bors[bot]2020-01-061-0/+4
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2714: Sanity check for missing server binary r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | | Sanity check for missing server binaryAleksey Kladov2019-12-311-0/+4
| | | |
* | | | Merge #2726bors[bot]2020-01-061-4/+25
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2726: Improve profiling output when duration filter is specified r=matklad a=michalt In particular: - Use strict inequality for comparisons, since that's what the filter syntax supports. - Convert to millis for comparisons, since that's the unit used both for the filter and when printing. Now something like `RA_PROFILE='*>0'` will only print things that took at least 1ms (when rounded to millis). Signed-off-by: Michal Terepeta <[email protected]> Co-authored-by: Michal Terepeta <[email protected]>
| * | | | Improve profiling output when duration filter is specifiedMichal Terepeta2020-01-021-4/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In particular: - Use strict inequality for comparisons, since that's what the filter syntax supports. - Convert to millis for comparisons, since that's the unit used both for the filter and when printing. Now something like `RA_PROFILE='*>0'` will only print things that took at least 1ms (when rounded to millis). Signed-off-by: Michal Terepeta <[email protected]>
* | | | | Merge #2751bors[bot]2020-01-061-17/+18
|\ \ \ \ \ | |_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2751: Crates up r=kjeremy a=kjeremy Co-authored-by: Jeremy Kolb <[email protected]>
| * | | | Crates upJeremy Kolb2020-01-061-17/+18
|/ / / /
* | | | Merge #2747bors[bot]2020-01-042-11/+19
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2747: BinOp helper to detect assignment r=matklad a=kjeremy Co-authored-by: Jeremy Kolb <[email protected]>
| * | | | BinOp helper to detect assignmentJeremy Kolb2020-01-042-11/+19
|/ / / /
* | | | Merge #2745bors[bot]2020-01-041-6/+0
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2745: remove unused file r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | remove unused fileAleksey Kladov2020-01-041-6/+0
| | | | |
* | | | | Merge #2743bors[bot]2020-01-0310-2510/+2328
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2743: Switch ast declaration from ron to a macro r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | Switch ast declaration from ron to a macroAleksey Kladov2020-01-039-2510/+2328
| | | | |
| * | | | kill unused fileAleksey Kladov2020-01-021-0/+0
| |/ / /
* | | | Merge #2742bors[bot]2020-01-035-10/+19
|\ \ \ \ | |_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2742: Split `infer` query into two for better profiling r=flodiebold a=michalt This is the same change as we did with `crate_def_map` and it does seem that we mostly spend time in salsa, without recomputing much on rust-analyzer side. Example output: ``` 233ms - handle_inlay_hints 163ms - get_inlay_hints 163ms - SourceAnalyzer::new 67ms - def_with_body_from_child_node 67ms - analyze_container 67ms - analyze_container 67ms - Module::from_definition 67ms - Module::from_file 67ms - crate_def_map 0ms - parse_macro_query (6 calls) 0ms - raw_items_query (1 calls) 66ms - ??? 0ms - crate_def_map (1 calls) 0ms - crate_def_map (1 calls) 96ms - infer 2ms - trait_solve_query (2 calls) 94ms - ??? 0ms - body_with_source_map_query (1 calls) 0ms - crate_def_map (1 calls) [...] ``` Signed-off-by: Michal Terepeta <[email protected]> Co-authored-by: Michal Terepeta <[email protected]>