Skip to content

Commit d950709

Browse files
committed
mappings: Change mappings to allow other items as item containers.
This is important for importing enum variants as items. gcc/rust/ChangeLog: * util/rust-hir-map.cc (Mappings::insert_ast_module): Rename to... (Mappings::insert_glob_container): ...this. (Mappings::lookup_ast_module): Rename to... (Mappings::lookup_glob_container): ...this. * util/rust-hir-map.h: Change declarations.
1 parent 48f93dc commit d950709

File tree

2 files changed

+12
-10
lines changed

2 files changed

+12
-10
lines changed

gcc/rust/util/rust-hir-map.cc

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1148,17 +1148,19 @@ Mappings::lookup_module_children (NodeId module)
11481148
}
11491149

11501150
void
1151-
Mappings::insert_ast_module (AST::Module *module)
1151+
Mappings::insert_glob_container (AST::Item *container)
11521152
{
1153-
rust_assert (modules.find (module->get_node_id ()) == modules.end ());
1154-
modules[module->get_node_id ()] = module;
1153+
rust_assert (glob_containers.find (container->get_node_id ())
1154+
== glob_containers.end ());
1155+
1156+
glob_containers[container->get_node_id ()] = container;
11551157
}
11561158

1157-
tl::optional<AST::Module *>
1158-
Mappings::lookup_ast_module (NodeId id)
1159+
tl::optional<AST::Item *>
1160+
Mappings::lookup_glob_container (NodeId id)
11591161
{
1160-
auto it = modules.find (id);
1161-
if (it == modules.end ())
1162+
auto it = glob_containers.find (id);
1163+
if (it == glob_containers.end ())
11621164
return tl::nullopt;
11631165

11641166
return {it->second};

gcc/rust/util/rust-hir-map.h

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -321,8 +321,8 @@ class Mappings
321321
void insert_visibility (NodeId id, Privacy::ModuleVisibility visibility);
322322
tl::optional<Privacy::ModuleVisibility &> lookup_visibility (NodeId id);
323323

324-
void insert_ast_module (AST::Module *);
325-
tl::optional<AST::Module *> lookup_ast_module (NodeId id);
324+
void insert_glob_container (AST::Item *);
325+
tl::optional<AST::Item *> lookup_glob_container (NodeId id);
326326
void insert_module_child (NodeId module, NodeId child);
327327
tl::optional<std::vector<NodeId> &> lookup_module_children (NodeId module);
328328

@@ -436,7 +436,7 @@ class Mappings
436436
std::map<NodeId, std::vector<NodeId>> module_child_map;
437437
std::map<NodeId, std::vector<Resolver::CanonicalPath>> module_child_items;
438438
std::map<NodeId, NodeId> child_to_parent_module_map;
439-
std::map<NodeId, AST::Module *> modules;
439+
std::map<NodeId, AST::Item *> glob_containers;
440440

441441
// AST mappings
442442
std::map<NodeId, AST::Item *> ast_item_mappings;

0 commit comments

Comments
 (0)