From 729ea41d145d369b203dca6f70b251ea365cb3d0 Mon Sep 17 00:00:00 2001 From: Derek Schuff Date: Fri, 6 Dec 2024 12:34:19 -0800 Subject: Add bulk-memory-opt feature and ignore call-indirect-overlong (#7139) LLVM recently split the bulk-memory-opt feature out from bulk-memory, containing just memory.copy and memory.fill. This change follows that, making bulk-memory-opt also enabled when all of bulk-memory is enabled. It also introduces call-indirect-overlong following LLVM, but ignores it, since Binaryen has always allowed the encoding (i.e. command line flags enabling or disabling the feature are accepted but ignored). --- src/passes/OptimizeInstructions.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'src/passes/OptimizeInstructions.cpp') diff --git a/src/passes/OptimizeInstructions.cpp b/src/passes/OptimizeInstructions.cpp index 881b7ea1f..792cf6235 100644 --- a/src/passes/OptimizeInstructions.cpp +++ b/src/passes/OptimizeInstructions.cpp @@ -1275,7 +1275,7 @@ struct OptimizeInstructions if (curr->type == Type::unreachable) { return; } - assert(getModule()->features.hasBulkMemory()); + assert(getModule()->features.hasBulkMemoryOpt()); if (auto* ret = optimizeMemoryCopy(curr)) { return replaceCurrent(ret); } @@ -1285,7 +1285,7 @@ struct OptimizeInstructions if (curr->type == Type::unreachable) { return; } - assert(getModule()->features.hasBulkMemory()); + assert(getModule()->features.hasBulkMemoryOpt()); if (auto* ret = optimizeMemoryFill(curr)) { return replaceCurrent(ret); } -- cgit v1.2.3