<div dir="auto"><div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">J. Gareth Moreton <<a href="mailto:gareth@moreton-family.com">gareth@moreton-family.com</a>> schrieb am Fr., 1. Nov. 2019, 12:56:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">So the tests all passed on i386-win32 and x86_64-win64, so that's s a <br>
good sign.  I can't submit the patches for evaluation yet because I <br>
haven't finished the design spec yet, and also because of a minor bug <br>
that deals with collapsing label clusters:<br>
<br>
     .p2align 4,,10<br>
     .p2align 3<br>
.Lj370:<br>
.Lj367:<br>
.Lj364:<br>
.Lj361:<br>
.Lj358:<br>
.Lj355:<br>
.Lj352:<br>
.Lj349:<br>
.Lj346:<br>
.Lj343:<br>
.Lj340:<br>
<br>
In this segment, everything is stripped except for the last label, which <br>
is fine as all the references are changed too. Unfortunately, the <br>
alignment fields are removed too, which shouldn't happen.  It doesn't <br>
produce incorrect code, but it may incur a performance penalty, so <br>
shouldn't be removed - I'm just trying to figure out why that's happening!<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Considering that you said that this feature is essentially cross platform: on some other platform the alignments might be important beside performance (e.g. a branch inside a branch delay slot or something like that). So, yeah, that should be fixed... </div><div dir="auto"><br></div><div dir="auto">Regards, </div><div dir="auto">Sven </div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
</blockquote></div></div></div>