[PATCH] D72047: Add an interface emitPrefix for MCCodeEmitter

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 1 01:10:51 PST 2020


craig.topper added a comment.

In D72047#1800301 <https://reviews.llvm.org/D72047#1800301>, @craig.topper wrote:

> My thought was just that the encoder only has 2 member variables in it so its quite small to create. And it would avoid exposing the concept of prefixes to the MCCodeEmitter interface. But lets see what other reviewers think.


But I guess we don't have a MCContext object in X86AsmBackend so we can't run the X86MCCodeEmitter constructor so maybe that doesn't work at all.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D72047/new/

https://reviews.llvm.org/D72047





More information about the llvm-commits mailing list