[llvm-commits] [PATCH][Hexagon] Clean immediate operand defintions

Sean Silva silvas at purdue.edu
Wed Nov 21 09:11:31 PST 2012


I was just about to ask for this on seeing the other commit :)

-  // For now, we use a generic print function for all operands.
-  let PrintMethod = "printNOneImmOperand";
[...]
+// Immediate operands.
+
+let PrintMethod = "printImmOperand" in {

you might want to preserve that "For now" comment.

-- Sean Silva

On Wed, Nov 21, 2012 at 11:46 AM, Jyotsna Verma <jverma at codeaurora.org> wrote:
> The attached patch changes the Hexagon specific immediate operand
> definitions to use one common let expression for PrintMethod. Please let me
> know if the patch is good to commit.



More information about the llvm-commits mailing list