[PATCH] D37262: The issues with X86 prefixes: step 2

Rafael Avila de Espindola via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 25 12:05:11 PDT 2017


Having a flag field that is used only on disassembly seems wrong.

Don't we support parsing our own output? I don't see trepne in any .s
test for example.

Cheers,
Rafael

Craig Topper via Phabricator <reviews at reviews.llvm.org> writes:

> craig.topper added a comment.
>
> I'm not sure I can approve growing the size of MCInst. Though I can't see anyway around it. @rafael what do you think?
>
>
> https://reviews.llvm.org/D37262


More information about the llvm-commits mailing list