[PATCH] D89735: [ms] [llvm-ml] Support purging macro definitions

Nico Weber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 2 16:26:48 PST 2020


thakis added a comment.

Can you add a test for the comma variant?



================
Comment at: llvm/lib/MC/MCParser/MasmParser.cpp:5579
+      break;
+    parseOptionalToken(AsmToken::EndOfStatement);
+  }
----------------
Why do we accept an optional EndOfStatement after every comma?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89735



More information about the llvm-commits mailing list