On Wed, Apr 8, 2009 at 3:30 PM, Jakob Stoklund Olesen <span dir="ltr"><<a href="mailto:stoklund@2pi.dk">stoklund@2pi.dk</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im">
On 08/04/2009, at 21.04, Dan Gohman wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
I agree that the code in MachineInstr::getNumExplicitOperands<br>
looks wrong. According to ARMInstrThumb.td, tPOP_RET has variable_ops<br>
though, and IIRC the way things currently work is that these<br>
extra operands get added after the implicit operands, so I think<br>
this patch isn't the right fix.<br>
</blockquote>
<br></div>
In that case, maybe this was the intention?<br>
Tested with "make check".</blockquote><div><br>Note that "make check" does not automatically rebuild the tools used in testing; you have to "make tools-only" first to make sure you're running the code you modified, rather than the binaries from a previous build.<br>
<br>I sent a patch to llvm-commits to solicit feedback as to whether it's safe -- see the archives if you want to try it out.<br></div></div><br>