[PATCH] D33278: [LLVM][x86][Inline Asm] - Enum support for MS syntax

[NAKAMURA Takumi via Phabricator via llvm-commits]

chapuni added a comment.

FYI, https://reviews.llvm.org/rL308966 has CRLF-ended paragraph in ParseIntelExpression().



================
Comment at: lib/Target/X86/AsmParser/X86AsmParser.cpp:1371
       } else {
-        InlineAsmIdentifierInfo &Info = SM.getIdentifierInfo();
+        InlineAsmIdentifierInfo Info;
         if (ParseIntelIdentifier(Val, Identifier, Info,
----------------
Is it initialized properly?


================
Comment at: lib/Target/X86/AsmParser/X86AsmParser.cpp:1592
+  // Check if the search yielded a constant integer (enum identifier).
+  if (Result && Info.isConstEnum()) {
+    // By creating MCConstantExpr we let the user of Val know it is safe
----------------
Info might be uninitialized here.


https://reviews.llvm.org/D33278