<div dir="ltr">To fix it we need to change offset8/offset16/etc to have two suboperands and update the printer to understand that. Also update the disassembler to add the segment to the MCInst when its creating it. When I did these two things the MCCodeEmitter broke because it tried to interpret the extra operand as an immediate. I think I probably need to change the form from RawFrm to something new that I can teach the MCCodeEmitter to handle correctly.<div>
<br></div><div>I haven't looked at the patches yet. I was out of town most of last week and trying to get caught back up.</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jan 14, 2014 at 11:11 PM, David Woodhouse <span dir="ltr"><<a href="mailto:dwmw2@infradead.org" target="_blank">dwmw2@infradead.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Tue, 2014-01-14 at 22:59 -0800, Craig Topper wrote:<br>
> I started trying to fix this tonight but it's pretty nasty to fix.<br>
> I'll try to make another go at it later this week.<br>
<br>
</div>Much of it seems fixed already; what's left to fix? The segment prefix<br>
override? Does that mean we get to fix disassembly of '0x65 0xa4' while<br>
we're at it? (Although we can't even *assemble* that one, I note.)<br>
<br>
Any comments on the patches I posted on Monday to fix various 16-bit<br>
disasm issues?<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
dwmw2<br>
<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br>~Craig
</div>