[PATCH] Add support for parsing the not operator in Microsoft inline assembly

Ehsan Akhgari ehsan.akhgari at gmail.com
Fri Jul 4 11:30:49 PDT 2014


================
Comment at: lib/Target/X86/AsmParser/X86AsmParser.cpp:412-413
@@ +411,4 @@
+        break;
+      case IES_PLUS:
+      case IES_MINUS:
+      case IES_NOT:
----------------
David Majnemer wrote:
> Can you please explain why you have `IES_PLUS` and `IES_MINUS` here?
IES_PLUS is the default value when we want to parse ~15 for example.  MSVC also accepts ~-15, but my patch doesn't support that properly I just realized, so I need to take the IES_MINUS out.

http://reviews.llvm.org/D4391






More information about the llvm-commits mailing list