[PATCH] D39968: [COFF] Ignore semicolons in module definition identifiers

[Rui Ueyama via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL319933: [COFF] Ignore semicolons in module definition identifiers (authored by ruiu).

Changed prior to commit:
  https://reviews.llvm.org/D39968?vs=123607&id=125772#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D39968

Files:
  llvm/trunk/lib/Object/COFFModuleDefinition.cpp
  llvm/trunk/test/tools/llvm-dlltool/coff-exports.def


Index: llvm/trunk/test/tools/llvm-dlltool/coff-exports.def
===================================================================
--- llvm/trunk/test/tools/llvm-dlltool/coff-exports.def
+++ llvm/trunk/test/tools/llvm-dlltool/coff-exports.def
@@ -5,11 +5,17 @@
 
 LIBRARY test.dll
 EXPORTS
-TestFunction
+TestFunction1
+TestFunction2;
+TestFunction3 ; This is a comment
 
 ; CHECK: File: test.dll
 ; CHECK: Format: COFF-import-file
 ; CHECK: Type: code
 ; CHECK: Name type: name
-; CHECK: Symbol: __imp_TestFunction
-; CHECK: Symbol: TestFunction
+; CHECK: Symbol: __imp_TestFunction1
+; CHECK: Symbol: TestFunction1
+; CHECK: Symbol: __imp_TestFunction2{{$}}
+; CHECK: Symbol: TestFunction2{{$}}
+; CHECK: Symbol: __imp_TestFunction3{{$}}
+; CHECK: Symbol: TestFunction3{{$}}
Index: llvm/trunk/lib/Object/COFFModuleDefinition.cpp
===================================================================
--- llvm/trunk/lib/Object/COFFModuleDefinition.cpp
+++ llvm/trunk/lib/Object/COFFModuleDefinition.cpp
@@ -117,7 +117,7 @@
       return Token(Identifier, S);
     }
     default: {
-      size_t End = Buf.find_first_of("=,\r\n \t\v");
+      size_t End = Buf.find_first_of("=,;\r\n \t\v");
       StringRef Word = Buf.substr(0, End);
       Kind K = llvm::StringSwitch<Kind>(Word)
                    .Case("BASE", KwBase)


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D39968.125772.patch
Type: text/x-patch
Size: 1318 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171206/62bf9d32/attachment.bin>