bjoernager/rust
bjoernager/rust
1
Fork 0
Code Issues Pull requests Activity

Use an empty TokenCursorFrame stack when capturing tokens

Browse source 
We will never need to pop  past our starting frame during token
capturing. Using an empty stack allows us to avoid pointless heap
allocations/deallocations.
This commit is contained in:
Aaron Hill 2021-01-08 18:16:20 -05:00
parent 26438b4738
commit 7b36408b5f
No known key found for this signature in database
GPG key ID: B4087E510E98B164
1 changed files with 9 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

10
compiler/rustc_parse/src/parser/mod.rs
View file

@ -1239,7 +1239,15 @@ impl<'a> Parser<'a> {
f: impl FnOnce(&mut Self) -> PResult<'a, R>,
) -> PResult<'a, (R, Option<LazyTokenStream>)> {
let start_token = (self.token.clone(), self.token_spacing);
let cursor_snapshot = self.token_cursor.clone();
let cursor_snapshot = TokenCursor {
frame: self.token_cursor.frame.clone(),
// We only ever capture tokens within our current frame,
// so we can just use an empty frame stack
stack: vec![],
desugar_doc_comments: self.token_cursor.desugar_doc_comments,
num_next_calls: self.token_cursor.num_next_calls,
append_unglued_token: self.token_cursor.append_unglued_token.clone(),
};
let ret = f(self)?;