[PATCH] D88337: [ms] [llvm-ml] Add support for .radix directive, and accept all radix specifiers

Eric Astor via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 25 13:16:29 PDT 2020


epastor created this revision.
epastor added a reviewer: thakis.
Herald added subscribers: llvm-commits, hiraditya.
Herald added a project: LLVM.
epastor requested review of this revision.

Add support for .radix directive, and radix specifiers [yY] (binary), [oOqQ] (octal), and [tT] (decimal).

Also, when lexing MASM integers, require radix specifier; MASM requires that all literals without a radix specifier be treated as in the default radix. (e.g., 0100 = 100)

Relanding D87400 <https://reviews.llvm.org/D87400>, now with fewer ms-inline-asm tests broken!


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D88337

Files:
  llvm/include/llvm/MC/MCParser/MCAsmLexer.h
  llvm/lib/MC/MCParser/AsmLexer.cpp
  llvm/lib/MC/MCParser/COFFMasmParser.cpp
  llvm/lib/MC/MCParser/MasmParser.cpp
  llvm/lib/Target/X86/AsmParser/X86AsmParser.cpp
  llvm/test/tools/llvm-ml/radix.test
  llvm/test/tools/llvm-ml/radix_errors.test
  llvm/tools/llvm-ml/llvm-ml.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D88337.294411.patch
Type: text/x-patch
Size: 15803 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200925/7ea22878/attachment.bin>


More information about the llvm-commits mailing list