commit f6866a2ffcc10a86a80bf6031d9edbcfed572d76
author Ben Clayton <bclayton@google.com> Mon Jan 11 22:02:42 2021 +0000
committer Commit Bot service account <commit-bot@chromium.org> Mon Jan 11 22:02:42 2021 +0000
tree 823a69abdeb4cc63e88fdae6263621a57cf28f31
parent 4527a512eb5ea12153a689b4583c3bef85a493de

Add std::hash<tint::Symbol> specialization

Allows symbols to be used as keys for std::unordered_map and std::unordered_set.
Replace all map / set use of uint32_t for Symbol, where applicable.

Change-Id: If142b4ad1f0ee65bc62209ae2f277e7746be19bb
Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/37262
Reviewed-by: dan sinclair <dsinclair@chromium.org>
Commit-Queue: dan sinclair <dsinclair@chromium.org>

src/type_determiner.cc

diff --git a/src/type_determiner.cc b/src/type_determiner.cc
index 0dc61d4..f16104d 100644
--- a/src/type_determiner.cc
+++ b/src/type_determiner.cc
@@ -124,7 +124,7 @@
     if (!func->IsEntryPoint()) {
       continue;
     }
-    for (const auto& callee : caller_to_callee_[func->symbol().value()]) {
+    for (const auto& callee : caller_to_callee_[func->symbol()]) {
       set_entry_points(callee, func->symbol());
     }
   }
@@ -133,9 +133,9 @@
 }
 
 void TypeDeterminer::set_entry_points(const Symbol& fn_sym, Symbol ep_sym) {
-  symbol_to_function_[fn_sym.value()]->add_ancestor_entry_point(ep_sym);
+  symbol_to_function_[fn_sym]->add_ancestor_entry_point(ep_sym);
 
-  for (const auto& callee : caller_to_callee_[fn_sym.value()]) {
+  for (const auto& callee : caller_to_callee_[fn_sym]) {
     set_entry_points(callee, ep_sym);
   }
 }
@@ -150,7 +150,7 @@
 }
 
 bool TypeDeterminer::DetermineFunction(ast::Function* func) {
-  symbol_to_function_[func->symbol().value()] = func;
+  symbol_to_function_[func->symbol()] = func;
 
   current_function_ = func;
 
@@ -389,7 +389,7 @@
       }
     } else {
       if (current_function_) {
-        caller_to_callee_[current_function_->symbol().value()].push_back(
+        caller_to_callee_[current_function_->symbol()].push_back(
             ident->symbol());
 
         auto* callee_func = mod_->FindFunctionBySymbol(ident->symbol());
@@ -906,7 +906,7 @@
     return true;
   }
 
-  auto iter = symbol_to_function_.find(symbol.value());
+  auto iter = symbol_to_function_.find(symbol);
   if (iter != symbol_to_function_.end()) {
     expr->set_result_type(iter->second->return_type());
     return true;