[PATCH] D131523: [ms] [llvm-ml] Add support for the (many) optional SEGMENT parameters

[Eric Astor via Phabricator via llvm-commits]

epastor added inline comments.


================
Comment at: llvm/lib/MC/MCParser/COFFMasmParser.cpp:349
+                         .CaseLower("const", SectionKind::getReadOnly())
+                         .Default(SectionKind::getData());
+  if (Kind.isText()) {
----------------
thakis wrote:
> Should we warn on unknown classes?
Strangely, I don't think so - my testing suggests that ML64.EXE treats any unknown class as equivalent to DATA.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D131523/new/

https://reviews.llvm.org/D131523