Attach `TokenStream` to `ast::Ty`

A `Ty` does not have outer attributes, so we only capture tokens
when parsing a `macro_rules!` matcher
This commit is contained in:
Aaron Hill 2020-08-21 18:18:04 -04:00
parent de4bd9f0f8
commit 1823dea7df
No known key found for this signature in database
GPG Key ID: B4087E510E98B164
12 changed files with 42 additions and 15 deletions

View File

@ -587,7 +587,7 @@ impl Pat {
_ => return None, _ => return None,
}; };
Some(P(Ty { kind, id: self.id, span: self.span })) Some(P(Ty { kind, id: self.id, span: self.span, tokens: None }))
} }
/// Walk top-down and call `it` in each place where a pattern occurs /// Walk top-down and call `it` in each place where a pattern occurs
@ -1169,7 +1169,7 @@ impl Expr {
_ => return None, _ => return None,
}; };
Some(P(Ty { kind, id: self.id, span: self.span })) Some(P(Ty { kind, id: self.id, span: self.span, tokens: None }))
} }
pub fn precedence(&self) -> ExprPrecedence { pub fn precedence(&self) -> ExprPrecedence {
@ -1867,6 +1867,7 @@ pub struct Ty {
pub id: NodeId, pub id: NodeId,
pub kind: TyKind, pub kind: TyKind,
pub span: Span, pub span: Span,
pub tokens: Option<TokenStream>,
} }
#[derive(Clone, Encodable, Decodable, Debug)] #[derive(Clone, Encodable, Decodable, Debug)]
@ -2145,7 +2146,7 @@ impl Param {
/// Builds a `Param` object from `ExplicitSelf`. /// Builds a `Param` object from `ExplicitSelf`.
pub fn from_self(attrs: AttrVec, eself: ExplicitSelf, eself_ident: Ident) -> Param { pub fn from_self(attrs: AttrVec, eself: ExplicitSelf, eself_ident: Ident) -> Param {
let span = eself.span.to(eself_ident.span); let span = eself.span.to(eself_ident.span);
let infer_ty = P(Ty { id: DUMMY_NODE_ID, kind: TyKind::ImplicitSelf, span }); let infer_ty = P(Ty { id: DUMMY_NODE_ID, kind: TyKind::ImplicitSelf, span, tokens: None });
let param = |mutbl, ty| Param { let param = |mutbl, ty| Param {
attrs, attrs,
pat: P(Pat { pat: P(Pat {
@ -2168,6 +2169,7 @@ impl Param {
id: DUMMY_NODE_ID, id: DUMMY_NODE_ID,
kind: TyKind::Rptr(lt, MutTy { ty: infer_ty, mutbl }), kind: TyKind::Rptr(lt, MutTy { ty: infer_ty, mutbl }),
span, span,
tokens: None,
}), }),
), ),
} }

View File

@ -451,7 +451,7 @@ pub fn noop_visit_ty_constraint<T: MutVisitor>(
} }
pub fn noop_visit_ty<T: MutVisitor>(ty: &mut P<Ty>, vis: &mut T) { pub fn noop_visit_ty<T: MutVisitor>(ty: &mut P<Ty>, vis: &mut T) {
let Ty { id, kind, span } = ty.deref_mut(); let Ty { id, kind, span, tokens: _ } = ty.deref_mut();
vis.visit_id(id); vis.visit_id(id);
match kind { match kind {
TyKind::Infer | TyKind::ImplicitSelf | TyKind::Err | TyKind::Never | TyKind::CVarArgs => {} TyKind::Infer | TyKind::ImplicitSelf | TyKind::Err | TyKind::Never | TyKind::CVarArgs => {}

View File

@ -1106,6 +1106,7 @@ impl<'a, 'hir> LoweringContext<'a, 'hir> {
id: node_id, id: node_id,
kind: TyKind::ImplTrait(impl_trait_node_id, bounds.clone()), kind: TyKind::ImplTrait(impl_trait_node_id, bounds.clone()),
span: constraint.span, span: constraint.span,
tokens: None,
}, },
itctx, itctx,
); );

View File

@ -61,6 +61,7 @@ pub fn expand_concat_idents<'cx>(
id: ast::DUMMY_NODE_ID, id: ast::DUMMY_NODE_ID,
kind: ast::TyKind::Path(None, ast::Path::from_ident(self.ident)), kind: ast::TyKind::Path(None, ast::Path::from_ident(self.ident)),
span: self.ident.span, span: self.ident.span,
tokens: None,
})) }))
} }
} }

View File

@ -607,6 +607,7 @@ impl DummyResult {
id: ast::DUMMY_NODE_ID, id: ast::DUMMY_NODE_ID,
kind: if is_error { ast::TyKind::Err } else { ast::TyKind::Tup(Vec::new()) }, kind: if is_error { ast::TyKind::Err } else { ast::TyKind::Tup(Vec::new()) },
span: sp, span: sp,
tokens: None,
}) })
} }
} }

View File

@ -54,7 +54,7 @@ impl<'a> ExtCtxt<'a> {
} }
pub fn ty(&self, span: Span, kind: ast::TyKind) -> P<ast::Ty> { pub fn ty(&self, span: Span, kind: ast::TyKind) -> P<ast::Ty> {
P(ast::Ty { id: ast::DUMMY_NODE_ID, span, kind }) P(ast::Ty { id: ast::DUMMY_NODE_ID, span, kind, tokens: None })
} }
pub fn ty_path(&self, path: ast::Path) -> P<ast::Ty> { pub fn ty_path(&self, path: ast::Path) -> P<ast::Ty> {

View File

@ -37,7 +37,8 @@ pub fn placeholder(
tokens: None, tokens: None,
}) })
}; };
let ty = || P(ast::Ty { id, kind: ast::TyKind::MacCall(mac_placeholder()), span }); let ty =
|| P(ast::Ty { id, kind: ast::TyKind::MacCall(mac_placeholder()), span, tokens: None });
let pat = let pat =
|| P(ast::Pat { id, kind: ast::PatKind::MacCall(mac_placeholder()), span, tokens: None }); || P(ast::Pat { id, kind: ast::PatKind::MacCall(mac_placeholder()), span, tokens: None });
@ -88,9 +89,12 @@ pub fn placeholder(
kind: ast::PatKind::MacCall(mac_placeholder()), kind: ast::PatKind::MacCall(mac_placeholder()),
tokens: None, tokens: None,
})), })),
AstFragmentKind::Ty => { AstFragmentKind::Ty => AstFragment::Ty(P(ast::Ty {
AstFragment::Ty(P(ast::Ty { id, span, kind: ast::TyKind::MacCall(mac_placeholder()) })) id,
} span,
kind: ast::TyKind::MacCall(mac_placeholder()),
tokens: None,
})),
AstFragmentKind::Stmts => AstFragment::Stmts(smallvec![{ AstFragmentKind::Stmts => AstFragment::Stmts(smallvec![{
let mac = P(ast::MacCallStmt { let mac = P(ast::MacCallStmt {
mac: mac_placeholder(), mac: mac_placeholder(),

View File

@ -270,6 +270,7 @@ pub fn nt_to_tokenstream(nt: &Nonterminal, sess: &ParseSess, span: Span) -> Toke
} }
Nonterminal::NtBlock(ref block) => block.tokens.clone(), Nonterminal::NtBlock(ref block) => block.tokens.clone(),
Nonterminal::NtPat(ref pat) => pat.tokens.clone(), Nonterminal::NtPat(ref pat) => pat.tokens.clone(),
Nonterminal::NtTy(ref ty) => ty.tokens.clone(),
Nonterminal::NtIdent(ident, is_raw) => { Nonterminal::NtIdent(ident, is_raw) => {
Some(tokenstream::TokenTree::token(token::Ident(ident.name, is_raw), ident.span).into()) Some(tokenstream::TokenTree::token(token::Ident(ident.name, is_raw), ident.span).into())
} }

View File

@ -28,7 +28,7 @@ pub(super) fn dummy_arg(ident: Ident) -> Param {
span: ident.span, span: ident.span,
tokens: None, tokens: None,
}); });
let ty = Ty { kind: TyKind::Err, span: ident.span, id: ast::DUMMY_NODE_ID }; let ty = Ty { kind: TyKind::Err, span: ident.span, id: ast::DUMMY_NODE_ID, tokens: None };
Param { Param {
attrs: AttrVec::default(), attrs: AttrVec::default(),
id: ast::DUMMY_NODE_ID, id: ast::DUMMY_NODE_ID,
@ -75,7 +75,12 @@ impl RecoverQPath for Ty {
Some(P(self.clone())) Some(P(self.clone()))
} }
fn recovered(qself: Option<QSelf>, path: ast::Path) -> Self { fn recovered(qself: Option<QSelf>, path: ast::Path) -> Self {
Self { span: path.span, kind: TyKind::Path(qself, path), id: ast::DUMMY_NODE_ID } Self {
span: path.span,
kind: TyKind::Path(qself, path),
id: ast::DUMMY_NODE_ID,
tokens: None,
}
} }
} }

View File

@ -510,7 +510,12 @@ impl<'a> Parser<'a> {
{ {
let span = self.prev_token.span.between(self.token.span); let span = self.prev_token.span.between(self.token.span);
self.struct_span_err(span, "missing trait in a trait impl").emit(); self.struct_span_err(span, "missing trait in a trait impl").emit();
P(Ty { kind: TyKind::Path(None, err_path(span)), span, id: DUMMY_NODE_ID }) P(Ty {
kind: TyKind::Path(None, err_path(span)),
span,
id: DUMMY_NODE_ID,
tokens: None,
})
} else { } else {
self.parse_ty()? self.parse_ty()?
}; };
@ -1046,7 +1051,7 @@ impl<'a> Parser<'a> {
// The user intended that the type be inferred, // The user intended that the type be inferred,
// so treat this as if the user wrote e.g. `const A: _ = expr;`. // so treat this as if the user wrote e.g. `const A: _ = expr;`.
P(Ty { kind: TyKind::Infer, span: id.span, id: ast::DUMMY_NODE_ID }) P(Ty { kind: TyKind::Infer, span: id.span, id: ast::DUMMY_NODE_ID, tokens: None })
} }
/// Parses an enum declaration. /// Parses an enum declaration.

View File

@ -141,7 +141,14 @@ impl<'a> Parser<'a> {
token::NtExpr(expr) token::NtExpr(expr)
} }
NonterminalKind::Literal => token::NtLiteral(self.parse_literal_maybe_minus()?), NonterminalKind::Literal => token::NtLiteral(self.parse_literal_maybe_minus()?),
NonterminalKind::Ty => token::NtTy(self.parse_ty()?), NonterminalKind::Ty => {
let (mut ty, tokens) = self.collect_tokens(|this| this.parse_ty())?;
// We have an eaten an NtTy, which could already have tokens
if ty.tokens.is_none() {
ty.tokens = Some(tokens);
}
token::NtTy(ty)
}
// this could be handled like a token, since it is one // this could be handled like a token, since it is one
NonterminalKind::Ident => { NonterminalKind::Ident => {
if let Some((ident, is_raw)) = get_macro_ident(&self.token) { if let Some((ident, is_raw)) = get_macro_ident(&self.token) {

View File

@ -626,6 +626,6 @@ impl<'a> Parser<'a> {
} }
pub(super) fn mk_ty(&self, span: Span, kind: TyKind) -> P<Ty> { pub(super) fn mk_ty(&self, span: Span, kind: TyKind) -> P<Ty> {
P(Ty { kind, span, id: ast::DUMMY_NODE_ID }) P(Ty { kind, span, id: ast::DUMMY_NODE_ID, tokens: None })
} }
} }