[PATCH] [X86][FastISel] Avoid introducing legacy SSE instructions if the subtarget has AVX.

Andrea Di Biagio Andrea_DiBiagio at sn.scee.net
Mon Feb 9 09:18:23 PST 2015


Hi Michael,

Here is a new version of the patch which hopefully addresses all your comments.
These are the main differences with respect to the previous patch:

1. I removed all the uses of IMPLICIT_DEF registers. If the target has AVX, the same register is passed to both input operands.
2. I changed the tests so that on SSE, we check that the instruction generated doesn't have a vex prefix.
3. I tried to factor out common code into a separate helper named 'X86SelectFPExtOrFPTrunc'.

Please let me know if ok to submit.

Thanks again for your time,
Andrea


http://reviews.llvm.org/D7438

Files:
  lib/Target/X86/X86FastISel.cpp
  test/CodeGen/X86/fast-isel-fptrunc-fpext.ll

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D7438.19587.patch
Type: text/x-patch
Size: 5130 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150209/6fa6fa3a/attachment.bin>


More information about the llvm-commits mailing list