[WebAssembly] Remove DEBUG_FUNCTION_NAME after llvm change

[sbc100]

Summary

Patch by Nicholas Wilson!

[ncw]

Thanks for splitting. It would be an extremely long chain of patches if I submitted them in chunks this small - I hope the chunks I'm using are roughly the right size for sending to you to (I could go bigger or smaller).

[sbc100]

As a rule I always prefer smaller focused changes, but at the same time it can be overwhelming to get so many changes to review at the same time.

I would say that especially for things like typos its nice to separate them since reviewing them is so trivial, and it leaves the real meaty changes with less noise in them.

test/wasm/archive.ll

entry:
   %call2 = call i32 @missing_func() #2
   ret i32 %call2
 }
 
 ; Verify that multually dependant object files in an archive is handled
 ; correctly.
 
 ; CHECK:      00000003 T _start
-; CHECK-NEXT: 00000003 T _start
 ; CHECK-NEXT: 00000001 T bar
-; CHECK-NEXT: 00000001 T bar
-; CHECK-NEXT: 00000002 T foo
 ; CHECK-NEXT: 00000002 T foo
 
 ; Verify that symbols from unused objects don't appear in the symbol table
 ; CHECK-NOT: hello
 
 ; Specifying the same archive twice is allowed.
 ; RUN: lld -flavor wasm %t.a %t.a %t.o -o %t.wasm

wasm/InputFiles.cpp

case WasmSymbol::SymbolType::GLOBAL_EXPORT: {
       } else {
         Segment->Discarded = true;
         LLVM_FALLTHROUGH; // Exclude global, and add the symbol as undefined
       }
     }
     case WasmSymbol::SymbolType::GLOBAL_IMPORT:
       S = createUndefined(WasmSym, Symbol::Kind::UndefinedGlobalKind);
       break;
-    case WasmSymbol::SymbolType::DEBUG_FUNCTION_NAME:
-      // These are for debugging only, no need to create linker symbols for them
-      continue;
     }
 
     Symbols.push_back(S);
     if (WasmSym.isFunction()) {
       FunctionSymbols[WasmSym.ElementIndex] = S;
       if (WasmSym.HasAltIndex)
         FunctionSymbols[WasmSym.AltIndex] = S;
     } else {