[MCParser] Support quoted section name for COFF

[MaskRay]

Summary

This features matches ELFAsmParser and makes it possible to use .section ".llvm.call-graph-profile","n"

[zequanwu]

LGTM.

[jhenderson]

llvm/test/MC/COFF/section.s

194

Maybe worth using some other characters in the name too, that wouldn't otherwise be allowed, e.g. '@', '{' etc. There's no real need for this to be a "standard" section name after all.

[MaskRay]

Improve test

llvm/lib/MC/MCParser/COFFAsmParser.cpp

bool COFFAsmParser::ParseSectionSwitch(StringRef Section,
 
   getStreamer().SwitchSection(getContext().getCOFFSection(
       Section, Characteristics, Kind, COMDATSymName, Type));
 
   return false;
 }
 
 bool COFFAsmParser::ParseSectionName(StringRef &SectionName) {
-  if (!getLexer().is(AsmToken::Identifier))
+  if (!getLexer().is(AsmToken::Identifier) && !getLexer().is(AsmToken::String))
     return true;
 
   SectionName = getTok().getIdentifier();
   Lex();
   return false;
 }
 
 // .section name [, "flags"] [, identifier [ identifier ], identifier]

llvm/test/MC/COFF/section.s

 // CHECK:          Characteristics [
 // CHECK-NEXT:       IMAGE_SCN_ALIGN_1BYTES
 // CHECK-NEXT:       IMAGE_SCN_CNT_INITIALIZED_DATA
 // CHECK-NEXT:       IMAGE_SCN_MEM_DISCARDABLE
 // CHECK-NEXT:       IMAGE_SCN_MEM_READ
 // CHECK-NEXT:     ]
 // CHECK:        }
 
+/// The section name can be quoted.
+.section "@#$-{","n"
+// CHECK:        Section {
+// CHECK-NEXT:     Number:
+// CHECK-NEXT:     Name: @#$-{
 
+// CHECK-NOT:    Section {
 // CHECK:      ]