aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | Merge #8364bors[bot]2021-04-068-33/+54
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8364: Memory usage improvements r=jonas-schievink a=alexmaco These are mostly focused on splitting up enum variants with large size differences between variants by `Box`-ing things up. In my testing this reduces the memory usage somewhere in the low percentages, even though the measurements are quite noisy. Co-authored-by: Alexandru Macovei <[email protected]>
| * | | | Avoid duplicating VfsPath in vfs::path_interner::PathInterner by using an ↵Alexandru Macovei2021-04-063-13/+18
| | | | | | | | | | | | | | | | | | | | IndexSet
| * | | | Use Box'es to reduce the size of hir_def::expr::Pat from 112 to 64 bytes on ↵Alexandru Macovei2021-04-064-8/+14
| | | | | | | | | | | | | | | | | | | | 64bit
| * | | | Use Box'es to reduce size of hir_def::expr::Expr from 128 to 72 bytes (on ↵Alexandru Macovei2021-04-063-12/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 64bit systems) Rationale: only a minority of variants used almost half the size. By keeping large members (especially in Option) behind a box the memory cost is only payed when the large variants are needed. This reduces the size Vec<Expr> needs to allocate.
* | | | | Merge #8368bors[bot]2021-04-0614-226/+246
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8368: Move Ty accessors to TyExt r=flodiebold a=Veykril CC #8313 Co-authored-by: Lukas Wirth <[email protected]>
| * | | | | Remove TyExt::dyn_trait_refLukas Wirth2021-04-061-10/+4
| | | | | |
| * | | | | Move Ty accessors to TyExtLukas Wirth2021-04-0614-226/+252
| | |_|/ / | |/| | |
* | | | | Merge #8355bors[bot]2021-04-0612-236/+272
|\ \ \ \ \ | |/ / / / |/| | / / | | |/ / | |/| | | | | | | | | | | | | | 8355: internal: do not drop errors from cargo metadata/check r=matklad a=matklad Work towards #3155 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | Sacrifice for the semicolon godAleksey Kladov2021-04-062-5/+5
| | | |
| * | | More robust status notificationsAleksey Kladov2021-04-0611-150/+165
| | | |
| * | | Prepare for more stateless status reportingAleksey Kladov2021-04-062-35/+36
| | | |
| * | | Use autoreload config to gate execution rather that stalenessAleksey Kladov2021-04-062-7/+8
| | | |
| * | | Avoid duplicate `cargo check`sAleksey Kladov2021-04-061-1/+12
| | | |
| * | | Improve readabilityAleksey Kladov2021-04-061-29/+28
| | | | | | | | | | | | | | | | | | | | It's better to order functions in the order they are called, if there's ordering between them.
| * | | internal: do not drop errors from cargo metadata/checkAleksey Kladov2021-04-064-51/+60
| | | | | | | | | | | | | | | | Work towards #3155
* | | | Merge #8366bors[bot]2021-04-0612-45/+109
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | 8366: Add chalk_ir::Const to TyKind::Array r=flodiebold a=Veykril CC #8313 Co-authored-by: Lukas Wirth <[email protected]>
| * | | Add chalk_ir::Const to TyKind::ArrayLukas Wirth2021-04-0612-45/+109
|/ / /
* | | Merge #8359bors[bot]2021-04-0613-42/+66
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | 8359: Add Lifetime to TyKind::Ref and DynTy r=flodiebold a=Veykril CC #8313 Co-authored-by: Lukas Wirth <[email protected]>
| * | Use a constructor function for Static lifetimesLukas Wirth2021-04-066-49/+37
| | |
| * | Always use Static lifetimes in chalk mappingLukas Wirth2021-04-061-1/+2
| | |
| * | Add Lifetime to DynTyLukas Wirth2021-04-063-1/+5
| | |
| * | Add Lifetime to TyKind::RefLukas Wirth2021-04-0613-43/+74
|/ /
* | Merge #8360bors[bot]2021-04-054-10/+23
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8360: Fix shifting of binders in FnPointer r=flodiebold a=flodiebold - don't shift in/out for Chalk mapping (we want to have the same binders now) - do shift in when creating the signature for a closure (though it shouldn't matter much) - do shift in when lowering a `fn()` type - correctly deal with the implied binder in TypeWalk Tested with the binders validator on various repos, so I'm pretty sure this doesn't mess things up :grimacing: Co-authored-by: Florian Diebold <[email protected]>
| * | Fix shifting of binders in FnPointerFlorian Diebold2021-04-054-10/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | - don't shift in/out for Chalk mapping (we want to have the same binders now) - do shift in when creating the signature for a closure (though it shouldn't matter much) - do shift in when lowering a `fn()` type - correctly deal with the implied binder in TypeWalk
* | | Merge #8358bors[bot]2021-04-059-40/+86
|\| | | | | | | | | | | | | | | | | | | | | | | 8358: Align FnPointer with Chalk r=flodiebold a=flodiebold CC #8313 Co-authored-by: Florian Diebold <[email protected]>
| * | Align FnPointer with ChalkFlorian Diebold2021-04-059-40/+86
|/ /
* | Merge #8356bors[bot]2021-04-0511-112/+108
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | 8356: Align more methods to Chalk r=flodiebold a=flodiebold Related to #8313. Move some inherent methods that don't exist in Chalk to an extension trait, remove some others. Co-authored-by: Florian Diebold <[email protected]>
| * | Substitution::prefix -> subst_prefixFlorian Diebold2021-04-054-11/+15
| | | | | | | | | | | | I probably want to get rid of this function completely later.
| * | Get rid of Substitution::suffixFlorian Diebold2021-04-052-10/+6
| | |
| * | Remove some unused methods, move some to types.rsFlorian Diebold2021-04-052-29/+15
| | |
| * | Binders::wrap_empty -> wrap_empty_bindersFlorian Diebold2021-04-053-16/+12
| | |
| * | Substitution::single -> from1Florian Diebold2021-04-054-14/+17
| | |
| * | Move ProjectionTy methods to extension traitFlorian Diebold2021-04-057-32/+43
|/ /
* | Merge #8353bors[bot]2021-04-054-18/+50
|\ \ | | | | | | | | | | | | | | | | | | | | | 8353: Replace hir_ty::Lifetime with chalk equivalent r=flodiebold a=Veykril Our `Lifetime` isn't really used yet so this is a rather simple change Co-authored-by: Lukas Wirth <[email protected]>
| * | Replace unused hir_ty::Lifetime with chalk equivalentsLukas Wirth2021-04-054-18/+50
| | |
* | | Merge #8352bors[bot]2021-04-051-9/+3
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8352: Remove dead legacy macro expansion code r=lnicola a=brandondong I was investigating some unrelated macro issue when I noticed this dead code. This legacy macro expansion logic was changed in https://github.com/rust-analyzer/rust-analyzer/pull/8128. Co-authored-by: Brandon <[email protected]>
| * | | Remove dead legacy macro expansion codeBrandon2021-04-051-9/+3
| | | |
* | | | Merge #8351bors[bot]2021-04-051-22/+21
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | 8351: Use more assoc. type aliases in the chalk interner r=flodiebold a=jonas-schievink Makes it sligthly easier to swap out these types Co-authored-by: Jonas Schievink <[email protected]>
| * | | Use more assoc. type aliases in the chalk internerJonas Schievink2021-04-051-22/+21
| | | | | | | | | | | | | | | | Makes it sligthly easier to swap out these types
* | | | Merge #8350bors[bot]2021-04-053-13/+24
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | 8350: internal: prepare to store OpQueue results in the queue itself r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | internal: prepare to store OpQueue results in the queue itselfAleksey Kladov2021-04-053-13/+24
| | | |
* | | | Merge #8348bors[bot]2021-04-0520-228/+296
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8348: Make `Binders` more like Chalk r=flodiebold a=flodiebold Working towards #8313. - hide `value` - use `VariableKinds` - adjust `subst` to be like Chalk's `substitute` - also clean up some other `TypeWalk` stuff to prepare for it being replaced by Chalk's `Fold` Co-authored-by: Florian Diebold <[email protected]>
| * | | | Rename shift_bound_vars{_out} to align with ChalkFlorian Diebold2021-04-055-25/+32
| | | | |
| * | | | Get rid of subst_bound_vars usesFlorian Diebold2021-04-053-5/+9
| | | | |
| * | | | Get rid of some walk_mut usesFlorian Diebold2021-04-052-10/+15
| | | | |
| * | | | Add Interner parameter to Binders::substituteFlorian Diebold2021-04-0514-43/+52
| | | | |
| * | | | Binders::subst -> substituteFlorian Diebold2021-04-0514-39/+39
| | | | |
| * | | | Use VariableKinds in BindersFlorian Diebold2021-04-057-84/+107
| | | | |
| * | | | Hide Binders internals moreFlorian Diebold2021-04-0511-88/+108
| | | | |
* | | | | Merge #8347bors[bot]2021-04-051-4/+15
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8347: Minor interning improvements r=jonas-schievink a=jonas-schievink bors r+ Co-authored-by: Jonas Schievink <[email protected]>
generated by cgit v1.2.3 (git 2.25.1) at 2024-11-29 23:50:51 +0000