[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