wasm-metadce: Improve name deduplication (#6138)

Avoid adding suffixes when we don't need them to keep names unique.

As background, the suffixes are not used by emcc at all, so they are just
for internal use in the tool. How that works is that metadce gets as input
the list of things the user cares about, with names for them, so it knows
the proper names to give imports and exports, and makes up names for
other things. Those made up names will not be read by the user, so we
can make them prettier as this PR does without breaking anything.

The main benefit of this PR is to make debugging easier. 

1 files changed, 18 insertions, 0 deletions

diff --git a/test/metadce/name_collision.wast b/test/metadce/name_collision.wast
new file mode 100644
index 000000000..62d11c697
--- /dev/null
+++ b/test/metadce/name_collision.wast
@@ -0,0 +1,18 @@
+(module
+  ;; This export is given the name "func$other" in the graph.text file, which
+  ;; collides with the internal name we give the function $other. A unique name
+  ;; should be generated for the function in the stdout that mentions it is
+  ;; unused, specifically
+  ;;
+  ;;  unused: func$other$0
+  ;;
+  ;; (the $0 suffix keeps it unique).
+  (export "test" (func $test))
+
+  ;; This function is used by the export.
+  (func $test)
+
+  ;; This function is not used, and as mentioned above it will be called
+  ;; func$other$0 in the output.
+  (func $other)
+)