[PATCH] Don't generate 64-bit movd after cmpneqsd in 32-bit mode (PR19059)

Craig Topper craig.topper at gmail.com
Mon Mar 10 19:17:17 PDT 2014


Can you move this

bool is64BitFP = (CMP00.getValueType() == MVT::f64);

to be below the AVX512 code since that path doesn't need it. Otherwise LGTM.


On Mon, Mar 10, 2014 at 6:21 PM, Hans Wennborg <hans at chromium.org> wrote:

>
>   On Mon, Mar 10, 2014 at 6:18 PM, Craig Topper <craig.topper at gmail.com>
> wrote:
>   > Is there an earlier check to ensure SSE2 is supported? Just want to
> make
>   > sure we have everything covered.
>
>   Yup, we basically check on entry to the CMPEQCombine function.
>
> http://llvm-reviews.chandlerc.com/D3009
>



-- 
~Craig
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140310/fb6a5e80/attachment.html>


More information about the llvm-commits mailing list