tree 0eedfcb440457672f2ffcbce7a1ceb9c61d68c0a
parent b79031308cc878488202beb99883ec1f2efd9a6d
author David 'Digit' Turner <digit@google.com> 1637929192 +0100
committer Commit Bot <commit-bot@chromium.org> 1638549209 +0000

Fix Label::hash() computation.

This CL fixes a subtle bug where the hash_ member of a Label
instance was not properly initialized after a call to ::Resolve()
in label.cc, and a few other trivial cases. This prevented the
proper use of std::unordered_map<Label, ...> types.

+ Slightly speed up Label::ComputeHash() by avoiding string hashing
  whenever possible, using the newly-introduced StringAtom::ptr_hash()
  method (which replicates std::hash<StringAtom>()()).

Measurements show no impact on performance or RAM usage.

Change-Id: I11b6645709cc8c8ff266bb5174c1bdd102919332
Reviewed-on: https://gn-review.googlesource.com/c/gn/+/12580
Commit-Queue: David Turner <digit@google.com>
Reviewed-by: Brett Wilson <brettw@chromium.org>