[PATCH] D41610: [X86] Implementation of X86Operand::print

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 29 14:07:22 PST 2017


RKSimon added inline comments.


================
Comment at: lib/Target/X86/AsmParser/X86Operand.h:115
+    return true;
+  }
+
----------------
If possible, I'd make this a lambda function inside the print function below and hopefully remove the need for the union.


https://reviews.llvm.org/D41610





More information about the llvm-commits mailing list