[PATCH] Handle tied sub-operands in AsmMatcherEmitter

Ulrich Weigand Ulrich.Weigand at de.ibm.com
Tue Apr 2 05:37:25 PDT 2013


Jakob Stoklund Olesen <stoklund at 2pi.dk> wrote on 28.03.2013 17:52:56:

> Thanks for the explanation. Jim understands this code better than I
> do, so I'll let him comment.
>
> I would like to see a solution that can be shared between ARM and PPC.

Agreed.  In principle, the solution I have implemented could be used
by ARM as well; they'd have to change their asm parser to handle
address operands piecewise, instead of via ParserMatchClass on the
whole operand.

If this is not feasible, and we need a solution even for that case,
I guess one way would be to allow "forward ties".  This means to
allow for a way to add operands to an MCInst "out of order",
which currently doesn't seem to possible: the addOperand method
always appends operands at the end ...

Bye,
Ulrich




More information about the llvm-commits mailing list