summaryrefslogtreecommitdiff
path: root/src/ir/ExpressionAnalyzer.cpp
diff options
context:
space:
mode:
authorAlon Zakai <azakai@google.com>2021-02-26 23:00:15 +0000
committerGitHub <noreply@github.com>2021-02-26 15:00:15 -0800
commit1ba4d259616a71a6a80662f1dff95fc125ecac2e (patch)
tree77ba8a3f52cc9c4e6a50a31fc17a395c70439f23 /src/ir/ExpressionAnalyzer.cpp
parentdb6fa71e32e67c85af03e3dab482fbd35ffd1889 (diff)
downloadbinaryen-1ba4d259616a71a6a80662f1dff95fc125ecac2e.tar.gz
binaryen-1ba4d259616a71a6a80662f1dff95fc125ecac2e.tar.bz2
binaryen-1ba4d259616a71a6a80662f1dff95fc125ecac2e.zip
Remove unneeded static_assert on number of Names in hashing (#3606)
The assertion is not really needed. Wasm64 will need changes to support more than 2^32 names, in theory, but (1) wasm64 is just memory64 atm, and (2) we'd need to add a general option for Index to be larger than 32 bits in general, so there is nothing specific to the hashing code here.
Diffstat (limited to 'src/ir/ExpressionAnalyzer.cpp')
-rw-r--r--src/ir/ExpressionAnalyzer.cpp2
1 files changed, 0 insertions, 2 deletions
diff --git a/src/ir/ExpressionAnalyzer.cpp b/src/ir/ExpressionAnalyzer.cpp
index fe4b4e80c..59f243e4d 100644
--- a/src/ir/ExpressionAnalyzer.cpp
+++ b/src/ir/ExpressionAnalyzer.cpp
@@ -350,8 +350,6 @@ size_t ExpressionAnalyzer::hash(Expression* curr) {
return;
}
rehash(digest, 2);
- static_assert(sizeof(Index) == sizeof(int32_t),
- "wasm64 will need changes here");
rehash(digest, internalNames[curr]);
}
void visitNonScopeName(Name curr) { rehash(digest, uint64_t(curr.str)); }