Add a name hint to getValidName() (#5653)

Without the hint, we always look for a valid name using name$0, $1, $2, etc.,
starting from 0, and in some cases that can lead to quadratic behavior.

Noticed on a testcase in the fuzzer that runs for over 24 seconds (I gave up at
that point) but takes only 2 seconds with this.

1 files changed, 6 insertions, 6 deletions

diff --git a/test/ctor-eval/results.wast.out b/test/ctor-eval/results.wast.out
index 0194cabb6..4259369b5 100644
--- a/test/ctor-eval/results.wast.out
+++ b/test/ctor-eval/results.wast.out
@@ -7,9 +7,9 @@
  (global $global3 (mut i32) (i32.const 13))
  (global $global4 (mut i32) (i32.const 14))
  (global $global5 (mut i32) (i32.const 15))
- (export "test1" (func $test1_0))
- (export "test3" (func $test3_0))
- (export "test5" (func $test5_0))
+ (export "test1" (func $test1_7))
+ (export "test3" (func $test3_8))
+ (export "test5" (func $test5_9))
  (export "keepalive" (func $5))
  (func $test1 (type $none_=>_none)
   (global.set $global1
@@ -66,13 +66,13 @@
    )
   )
  )
- (func $test1_0 (type $none_=>_none)
+ (func $test1_7 (type $none_=>_none)
   (nop)
  )
- (func $test3_0 (type $none_=>_i32) (result i32)
+ (func $test3_8 (type $none_=>_i32) (result i32)
   (i32.const 42)
  )
- (func $test5_0 (type $none_=>_i32) (result i32)
+ (func $test5_9 (type $none_=>_i32) (result i32)
   (call $import)
   (i32.const 100)
  )