[all-commits] [llvm/llvm-project] 6b70a8: [ms] [llvm-ml] Add support for .radix directive, a...

Eric Astor via All-commits all-commits at lists.llvm.org
Tue Sep 29 13:56:49 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 6b70a83d9cc0ec17aa4bc199081c0a51e65be6dd
      https://github.com/llvm/llvm-project/commit/6b70a83d9cc0ec17aa4bc199081c0a51e65be6dd
  Author: Eric Astor <epastor at google.com>
  Date:   2020-09-29 (Tue, 29 Sep 2020)

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

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

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, now with fewer ms-inline-asm tests broken!

Reviewed By: rnk

Differential Revision: https://reviews.llvm.org/D88337




More information about the All-commits mailing list