From 1ba4d259616a71a6a80662f1dff95fc125ecac2e Mon Sep 17 00:00:00 2001 From: Alon Zakai Date: Fri, 26 Feb 2021 23:00:15 +0000 Subject: 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. --- src/ir/ExpressionAnalyzer.cpp | 2 -- 1 file changed, 2 deletions(-) (limited to 'src/ir/ExpressionAnalyzer.cpp') 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)); } -- cgit v1.2.3