[PATCH] D37412: [X86][MS-InlineAsm] Extended support for variables / identifiers on memory / immediate expressions

[coby via Phabricator via llvm-commits]

coby added inline comments.


================
Comment at: lib/Target/X86/AsmParser/X86AsmParser.cpp:589
+          return false;
+        }
+      // Treat a symbolic constant like an integer
----------------
RKSimon wrote:
> RKSimon wrote:
> > ```
> > if (IDInfo.isKind(InlineAsmIdentifierInfo::EnumValKind))
> >   return onInteger(IDInfo.Enum.EnumVal, ErrMsg);
> > ```
> This tidyup would still be nice.
sorry, don't know why've i missed this one out
will be added to the commit
thanks


================
Comment at: lib/Target/X86/AsmParser/X86AsmParser.cpp:595
+        return false;
+      }
       PrevState = State;
----------------
RKSimon wrote:
> RKSimon wrote:
> > ```
> > // Treat a symbolic constant like an integer
> > if (auto *CE = dyn_cast<MCConstantExpr>(SymRef))
> >   return onInteger(CE->getValue(), ErrMsg);
> > ```
> This tidyup would still be nice.
same


Repository:
  rL LLVM

https://reviews.llvm.org/D37412