<br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Nov 28, 2012 at 1:00 PM, Jim Grosbach <span dir="ltr"><<a href="mailto:grosbach@apple.com" target="_blank">grosbach@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Rafael,<br>
<br>
One question below:<br>
<div><div class="h5"><br>
> +    if (Dst.isReg()) {<br>
> +      assert(Dst.getReg() == MachineLocation::VirtualFP);<br>
> +      assert(!Src.isReg());<br>
> +      MCCFIInstruction Inst =<br>
> +        MCCFIInstruction::createDefCfa(Label, Src.getReg(), -Src.getOffset());<br>
<br>
</div></div>The assert(!Src.isReg()) immediately followed by Src.getReg() seems odd. Is it correct? If so, can you add some comments and maybe a message to the asserts explaining what's actually going wrong if they fire?<br>
<br></blockquote><div><br></div><div>Agreed, I was looking at it earlier and couldn't figure out what was meant in 30 seconds or so. Some comments would be great :)</div><div><br></div><div>-eric </div></div></div>