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

Nico Weber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 14 10:27:49 PDT 2020


thakis added inline comments.


================
Comment at: llvm/lib/MC/MCParser/AsmLexer.cpp:397
+    CurPtr = findLastDigit(CurPtr, DefaultRadix);
+    if (*CurPtr == '.' || *CurPtr == 'e' || *CurPtr == 'E') {
+      if (*CurPtr == '.')
----------------
Can't hex literals contain 'e' in non-float literals as well? how is that handled?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87400



More information about the llvm-commits mailing list