[PATCH] D42193: [WebAssembly] Remove DEBUG_FUNCTION_NAME after llvm change

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 17 11:35:22 PST 2018


sbc100 created this revision.
Herald added subscribers: llvm-commits, sunfish, JDevlieghere, aheejin, jgravelle-google, dschuff, jfb.
sbc100 added a reviewer: ncw.

Patch by Nicholas Wilson!


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D42193

Files:
  test/wasm/archive.ll
  wasm/InputFiles.cpp


Index: wasm/InputFiles.cpp
===================================================================
--- wasm/InputFiles.cpp
+++ wasm/InputFiles.cpp
@@ -252,9 +252,6 @@
     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);
Index: test/wasm/archive.ll
===================================================================
--- test/wasm/archive.ll
+++ test/wasm/archive.ll
@@ -27,10 +27,7 @@
 ; 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


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D42193.130235.patch
Type: text/x-patch
Size: 933 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180117/62767942/attachment-0001.bin>


More information about the llvm-commits mailing list