[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