Shrink ast::{Generics,Impl,Block}#101472
Shrink ast::{Generics,Impl,Block}#101472nnethercote wants to merge 4 commits intorust-lang:masterfrom
ast::{Generics,Impl,Block}#101472Conversation
|
@bors try @rust-timer queue |
|
Awaiting bors try build completion. @rustbot label: +S-waiting-on-perf |
|
⌛ Trying commit 57eb7e7 with merge 0fe233aee325ca40218db024909c8b834c97dba0... |
|
The job Click to see the possible cause of the failure (guessed by this bot) |
|
💔 Test failed - checks-actions |
|
The job Click to see the possible cause of the failure (guessed by this bot) |
| #[derive(Clone, Encodable, Decodable, Debug)] | ||
| pub struct EnumDef { | ||
| pub variants: Vec<Variant>, | ||
| pub variants: Box<[Variant]>, |
There was a problem hiding this comment.
Maybe we could use std::boxed::ThinBox or something similar to further reduce the size from 16 to 8 bytes? By storing the slice size on the heap.
There was a problem hiding this comment.
There is also thin_vec::ThinVec, which I introduced to the compiler in #100869. Unfortunately, getting any of these kinds of changes to show a clear performance benefit has been hard.
|
This one seems unlikely to produce any wins. |
r? @ghost