[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