<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">2013/1/9 Loren Merritt <span dir="ltr"><<a href="mailto:git@videolan.org" target="_blank">git@videolan.org</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
x264 | branch: master | Loren Merritt <<a href="mailto:pengvado@akuvian.org">pengvado@akuvian.org</a>> | Sun Dec  2 15:56:30 2012 +0000| [f6c628650558803ed65cb15c1853113cc589ae4a] | committer: Jason Garrett-Glaser<br>

<br>
x86inc: activate REP_RET automatically<br>
Now RET checks whether it immediately follows a branch, so the programmer dosen't have to keep track of that condition.<br>
REP_RET is still needed manually when it's a branch target, but that's much rarer.<br>
The implementation involves lots of spurious labels, but that's ok because we strip them.<br>
<br>
> <a href="http://git.videolan.org/gitweb.cgi/x264.git/?a=commit;h=f6c628650558803ed65cb15c1853113cc589ae4a" target="_blank">http://git.videolan.org/gitweb.cgi/x264.git/?a=commit;h=f6c628650558803ed65cb15c1853113cc589ae4a</a><br>

---<br>
<br>
 common/x86/cabac-a.asm   |    4 ++--<br>
 common/x86/dct-a.asm     |    4 ++--<br>
 common/x86/deblock-a.asm |   28 ++++++++++++++--------------<br>
 common/x86/mc-a.asm      |   44 ++++++++++++++++++++++----------------------<br>
 common/x86/mc-a2.asm     |   46 +++++++++++++++++++++++-----------------------<br>
 common/x86/predict-a.asm |   32 ++++++++++++++++----------------<br>
 common/x86/quant-a.asm   |   16 ++++++++--------<br>
 common/x86/x86inc.asm    |   36 +++++++++++++++++++++++++++++++-----<br>
 tools/checkasm-a.asm     |    2 +-<br>
 9 files changed, 119 insertions(+), 93 deletions(-)<br><br>
</blockquote></div><br>Do we still need this hack? Phenom prefers "ret $0" over "rep ret" and Bulldozer and all Intel CPUs seem to require neither.<br clear="all"><div><br></div>-- <br>Zuxy<br>Beauty is truth,<br>
While truth is beauty.<br>PGP KeyID: E8555ED6
</div></div>