diff options
| author | Philip Herron <philip.herron@embecosm.com> | 2021-01-08 17:18:00 +0000 |
|---|---|---|
| committer | Philip Herron <herron.philip@googlemail.com> | 2021-01-10 15:59:22 +0000 |
| commit | 9a3ad294e0af5ed45b53c340382a7806fb150bdf (patch) | |
| tree | 81458d62a2191b38be775026f2f8583d3ca7c2ee /gcc/rust/resolve/rust-ast-resolve-pattern.h | |
| parent | 0f42a240e53e932de0ae4799d54fe0bd15d06047 (diff) | |
| download | gcc-9a3ad294e0af5ed45b53c340382a7806fb150bdf.zip gcc-9a3ad294e0af5ed45b53c340382a7806fb150bdf.tar.gz gcc-9a3ad294e0af5ed45b53c340382a7806fb150bdf.tar.bz2 | |
Add in a check for unused decls within ribs.
This lead to cleanup of the name resolver as the usage of mappings means
that in a given rib if there are no references to a decl NodeId that
means it was not used.
To get on par with the offical rust compiler it should be allowed to have
a reference where the name was assigned but not used but this might be a seperate pass.
Diffstat (limited to 'gcc/rust/resolve/rust-ast-resolve-pattern.h')
| -rw-r--r-- | gcc/rust/resolve/rust-ast-resolve-pattern.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/gcc/rust/resolve/rust-ast-resolve-pattern.h b/gcc/rust/resolve/rust-ast-resolve-pattern.h index fc2da70..c79f7d3 100644 --- a/gcc/rust/resolve/rust-ast-resolve-pattern.h +++ b/gcc/rust/resolve/rust-ast-resolve-pattern.h @@ -81,7 +81,8 @@ public: // if we have a duplicate id this then allows for shadowing correctly // as new refs to this decl will match back here so it is ok to overwrite resolver->get_name_scope ().insert (pattern.get_ident (), - pattern.get_node_id ()); + pattern.get_node_id (), + pattern.get_locus ()); resolver->insert_new_definition (pattern.get_node_id (), Definition{pattern.get_node_id (), parent}); |