[COFF] Ignore semicolons in module definition identifiers

[dmajor]

Summary

The NSS project's .def files make heavy use of semicolons in a frightening attempt at portability: https://hg.mozilla.org/projects/nss/raw-file/tip/lib/ckfw/capi/nsscapi.def

lld-link was treating the semicolon as part of the export name, resulting in unresolved symbols. This patch includes ';' in the list of characters to split on.

[ruiu]

The NSS project's .def files make heavy use of semicolons in a frightening attempt at portability: https://hg.mozilla.org/projects/nss/raw-file/tip/lib/ckfw/capi/nsscapi.def

Ooh, that's an amazing piece of super hacky code. It's even funny. :) It feels like it would probably be much easier to prepare separate file for each platform.

Anyway, I believe this patch is fine, but can you add a testcase?

[dmajor]

Added a test. If this looks OK, do you mind landing it for me?

[dmajor]

@ruiu Does this patch look ok?

lib/Object/COFFModuleDefinition.cpp

case ',':
       Buf = Buf.drop_front();
       return Token(Comma, ",");
     case '"': {
       StringRef S;
       std::tie(S, Buf) = Buf.substr(1).split('"');
       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)
                    .Case("CONSTANT", KwConstant)
                    .Case("DATA", KwData)
                    .Case("EXPORTS", KwExports)
                    .Case("HEAPSIZE", KwHeapsize)
                    .Case("LIBRARY", KwLibrary)