Do not emit the DataCount section (#2010)

Its presence was causing validation errors in the Emscripten test
suite. This should be reverted once the default feature set is no
longer All.

1 files changed, 8 insertions, 3 deletions

diff --git a/src/wasm/wasm-binary.cpp b/src/wasm/wasm-binary.cpp
index f3642203d..d6091e93c 100644
--- a/src/wasm/wasm-binary.cpp
+++ b/src/wasm/wasm-binary.cpp
@@ -317,9 +317,14 @@ void WasmBinaryWriter::writeDataCount() {
   if (!wasm->features.hasBulkMemory() || !wasm->memory.segments.size()) {
     return;
   }
-  auto start = startSection(BinaryConsts::Section::DataCount);
-  o << U32LEB(wasm->memory.segments.size());
-  finishSection(start);
+
+  // TODO(tlively): re-enable writing the data count once the default feature
+  // set is no longer All, which causes validation errors in Emscripten due to
+  // the presence of an unrecognized section.
+
+  // auto start = startSection(BinaryConsts::Section::DataCount);
+  // o << U32LEB(wasm->memory.segments.size());
+  // finishSection(start);
 }
 
 void WasmBinaryWriter::writeDataSegments() {