commit b9e0e13d19326d3bbdb382232e120be2e83d1ec6
author Steven Perron <stevenperron@google.com> Mon Jan 10 14:24:44 2022 +0000
committer GitHub <noreply@github.com> Mon Jan 10 09:24:44 2022 -0500
tree 5a7bdd8c5b49705169a6a3b7d0dd7ee9cd220dcb
parent c5ee1bc7bd7c8aab529e22925eafe633f8807759

Remove misleading comment. (#4676)

The comment in `Array::GetExtraHashWords` is misleading because getting
the hash words is split up between the generic `Type::GetHashWords` and
the type specific `Type::GetExtraHashWords`.  While `IsSameImpl` is
self-contained.  Removing the comment since it is misleading and no
comment is really needed.

Fixes #3248

source/opt/types.cpp

diff --git a/source/opt/types.cpp b/source/opt/types.cpp
index b1eb3a5..ea4baad 100644
--- a/source/opt/types.cpp
+++ b/source/opt/types.cpp
@@ -425,7 +425,6 @@
 void Array::GetExtraHashWords(std::vector<uint32_t>* words,
                               std::unordered_set<const Type*>* seen) const {
   element_type_->GetHashWords(words, seen);
-  // This should mirror the logic in IsSameImpl
   words->insert(words->end(), length_info_.words.begin(),
                 length_info_.words.end());
 }