[PATCH] D104964: [ms] [llvm-ml] Add support for numeric built-in symbols

[Eric Astor via Phabricator via llvm-commits]

epastor added a comment.

In D104964#2853790 <https://reviews.llvm.org/D104964#2853790>, @thakis wrote:

> Can you check if ml.exe allows defining a local variable overriding e.g. `@Line`? What happens then? Does it error? Can you add a test for that?

Confirmed that ml.exe errors in this case; matched this behavior, and added corresponding test.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104964