Store a `Symbol` instead of an `Ident` in `VariantDef`/`FieldDef` by Aaron1011 ·...

 3 months ago
source link: https://github.com/rust-lang/rust/pull/92533
Go to the source link to view the article. You can view the picture content, updated content and better typesetting reading experience. If the link is damaged, please click the button below to view the snapshot at that time.

Copy link


Aaron1011 commented 17 days ago

edited by RalfJung

The field is also renamed from ident to name. In most cases,
we don't actually need the Span. A new ident method is added
to VariantDef and FieldDef, which constructs the full Ident
using tcx.def_ident_span(). This method is used in the cases
where we actually need an Ident.

This makes incremental compilation properly track changes
to the Span, without all of the invalidations caused by storing
a Span directly via an Ident.

About Joyk

Aggregate valuable and interesting links.
Joyk means Joy of geeK