diff --git a/src/libsyntax/print/pp.rs b/src/libsyntax/print/pp.rs index d8a8cbb655b4b..45eb6995a7699 100644 --- a/src/libsyntax/print/pp.rs +++ b/src/libsyntax/print/pp.rs @@ -300,6 +300,8 @@ impl Default for BufEntry { } } +const SPACES: [u8; 128] = [b' '; 128]; + impl<'a> Printer<'a> { pub fn last_token(&mut self) -> Token { self.buf[self.right].token.clone() @@ -580,10 +582,24 @@ impl<'a> Printer<'a> { debug!("print String({})", s); // assert!(len <= space); self.space -= len; - while self.pending_indentation > 0 { - write!(self.out, " ")?; - self.pending_indentation -= 1; + + // Write the pending indent. A more concise way of doing this would be: + // + // write!(self.out, "{: >n$}", "", n = self.pending_indentation as usize)?; + // + // But that is significantly slower than using `SPACES`. This code is + // sufficiently hot, and indents can get sufficiently large, that the + // difference is significant on some workloads. + let spaces_len = SPACES.len() as isize; + while self.pending_indentation >= spaces_len { + self.out.write_all(&SPACES)?; + self.pending_indentation -= spaces_len; } + if self.pending_indentation > 0 { + self.out.write_all(&SPACES[0..self.pending_indentation as usize])?; + self.pending_indentation = 0; + } + write!(self.out, "{}", s) }