<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Aug 26, 2014 at 10:04 AM, Zinovy Nis <span dir="ltr"><<a href="mailto:zinovy.nis@gmail.com" target="_blank">zinovy.nis@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div id=":aap" class="" style="overflow:hidden">Sorry for the late comment, but why we need<br>

<br>
```if (Subtarget->is64Bit()) {```<br>
<br>
check? As I remember, instructions like PMOVSXWD are supported even in 32-bit mode?<br>
<br>
I'm asking because I have a significant regression on EEMBC1.1 for autocorrelation tests due to lots of insert-like generated instructions for SLM target under -m32. Removing this check fixes the regression.</div></blockquote>
</div></div><br><div>The idea behind this was just flawed. It was intending to make sure we could load 64-bits.... but that is neither what the check means nor necessary! The code handles 32-bit architectures just fine as you point out. =] Fixed in r216538.</div>
</div>