[PATCH] D95375: [MC] Adding MCContext parameter to MCAsmBackend::relaxInstruction
David Ferenc Szabo via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 25 10:58:09 PST 2021
DavidFerencSzabo added a comment.
@MaskRay No. Its a Machine Instruction Bundle. (https://lists.llvm.org/pipermail/llvm-dev/2011-December/045851.html).
But reading this I stumbled upon this line
> There is no need to add the equivalent of MI bundle to MCInst. A MI bundle should be concatenated into a single MCInst by storing opcodes as integer operands. e.g.
This whole change might be unneded and the idiomatic way to do this is concatenation?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D95375/new/
https://reviews.llvm.org/D95375
More information about the llvm-commits
mailing list