<div dir="ltr">Hi Sergey,<div><br></div><div>Thanks for your update!</div><div><br></div>Unfortunately, with the latest patch, I can see "miscompare" failures for 445.gobmk and 482.sphinx3 in spec2006.<div><br></div>
<div>Can you verify it in your local box?</div><div><br></div><div>I think you can "run" them with qemu following <a href="http://lists.cs.uiuc.edu/pipermail/llvmdev/2014-April/072085.html">http://lists.cs.uiuc.edu/pipermail/llvmdev/2014-April/072085.html</a>, if you don't really have aarch64 hardware.</div>
<div><br></div><div>Thanks,</div><div>-Jiangning</div><div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2014-08-13 16:51 GMT+08:00 Sergey Dmitrouk <span dir="ltr"><<a href="mailto:sdmitrouk@accesssoftek.com" target="_blank">sdmitrouk@accesssoftek.com</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Jiangning,<br>
<div class=""><br>
> The 2nd compare instruction should not be under the same condition as the<br>
> 1st one, I think.<br>
<br>
</div>It should for currently supported transformations, otherwise we can try<br>
to adjust wrong instruction.  Allowing second compare be followed by<br>
arbitrary instruction is a separate case that needs changes in at least<br>
two places (need to take into account that we can't touch it).  I can do<br>
this if you want to have it now, but it'll increase size of the patch<br>
even more, so I just added TODO item for it at the top for now.<br>
<div class=""><br>
>    Can you try attached small case? With your patch, it may trigger<br>
>    segmentation fault.<br>
<br>
</div>I missed handling of "cmp; ...; b.??" in adjustCmp(), it assumed the<br>
next instruction to be Bcc.  Thank you, I reduced that code and added<br>
it to tests.  I also added hidden command-line option to enable/disable<br>
the pass to omit recompilation just to disable it.<br>
<br>
Thanks,<br>
Sergey<br>
<div class="HOEnZb"><div class="h5"><br>
On Tue, Aug 12, 2014 at 06:20:20PM -0700, Jiangning Liu wrote:<br>
>    Hi Sergey,<br>
>    The 2nd compare instruction should not be under the same condition as the<br>
>    1st one, I think.<br>
>    Can you try attached small case? With your patch, it may trigger<br>
>    segmentation fault.<br>
>    Thanks,<br>
>    -Jiangning<br>
</div></div></blockquote></div><br></div>