Fuzzer: Use NameTypes in RoundTrip (#3986)

This works around the issue with wasm gc types sometimes getting
truncated (as the default names can be very long or even infinitely
recursive). If the truncation leads to name collision, the wast is not
valid.

1 files changed, 5 insertions, 1 deletions

diff --git a/scripts/fuzz_opt.py b/scripts/fuzz_opt.py
index eff4baeef..f4c260665 100755
--- a/scripts/fuzz_opt.py
+++ b/scripts/fuzz_opt.py
@@ -886,7 +886,11 @@ class RoundtripText(TestCaseHandler):
     frequency = 0.05
 
     def handle(self, wasm):
-        run([in_bin('wasm-dis'), wasm, '-o', 'a.wast'])
+        # use name-types because in wasm GC we can end up truncating the default
+        # names which are very long, causing names to collide and the wast to be
+        # invalid
+        # FIXME: run name-types by default during load?
+        run([in_bin('wasm-opt'), wasm, '--name-types', '-S', '-o', 'a.wast'] + FEATURE_OPTS)
         run([in_bin('wasm-opt'), 'a.wast'] + FEATURE_OPTS)