On behalf of Matthew Wahab:
authorEvgeny Astigeevich <evgeny.astigeevich@arm.com>
Tue, 16 Dec 2014 18:16:17 +0000 (18:16 +0000)
committerEvgeny Astigeevich <evgeny.astigeevich@arm.com>
Tue, 16 Dec 2014 18:16:17 +0000 (18:16 +0000)
commitb42003d2bf015df6417cb3a60ec6a5fee8cc0279
tree43e2ebf580b9deea51773c58667d7964bc9d1b49
parent0dc19792937bb64d795a9ff809eb73cc3fb81fd0
On behalf of Matthew Wahab:

An instruction alias defined with InstAlias and an optional operand in the
middle of the AsmString field, "..${a} <operands>", would get the final
"}" printed in the instruction disassembly. This wouldn't happen if the optional
operand appeared as the last item in the AsmString which is how the current
backends avoided the problem.

There don't appear to be any tests for this part of Tablegen but it passes the
pre-commit tests. Manually tested the change by enabling the generic alias
printer in the ARM backend and checking the output.

Differential Revision: http://reviews.llvm.org/D6529

llvm-svn: 224348
llvm/utils/TableGen/AsmWriterEmitter.cpp