<div dir="ltr">Oh - I remember now: Branch relaxation only runs during object emission. If we waited until relaxation to fix the shadows, then the shadow optimization wouldn't run when you're outputting assembly files. The idea that you could get different output by passing -S vs -c seemed too non-obvious, so I came up with the current solution.<div>
<br><div><div>Cheers,</div><div>Lang.</div></div></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jul 24, 2014 at 4:39 PM, Lang Hames <span dir="ltr"><<a href="mailto:lhames@gmail.com" target="_blank">lhames@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Joerg,<div><br></div><div>I believe there is a perfectly good reason that I didn't use the branch relaxation code for this. I've just completely forgotten what it is. (Cards on the table - I wrote this patch a while back and have been sitting on it while I dealt with some other JIT bugs.)</div>

<div><br></div><div>Let me go figure it out again...</div><div><br></div><div>Cheers,</div><div>Lang.</div><div><br></div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><br><div class="gmail_quote">
On Thu, Jul 24, 2014 at 4:33 PM, Joerg Sonnenberger <span dir="ltr"><<a href="mailto:joerg@britannica.bec.de" target="_blank">joerg@britannica.bec.de</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>On Thu, Jul 24, 2014 at 03:49:05PM -0700, Eric Christopher wrote:<br>
> So you've create an EmitAndCountInstruction that you use for<br>
> everything except nop... since it looks like this stuff is more<br>
> multiplatform than just x86, why not generalize this out to the<br>
> AsmPrinter level (other than the fact that x86 is the only variable<br>
> length instruction set architecture we have?). Also, why not nop?<br>
<br>
</div>In fact, we already have logic for this -- the branch relaxation code.<br>
<span><font color="#888888"><br>
Joerg<br>
</font></span><div><div>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>