Fwd: [PATCH] Model sqrtss as a binary operation with one source operand tied to the destination (PR14221)

Sanjay Patel spatel at rotateright.com
Wed Nov 19 15:02:42 PST 2014


---------- Forwarded message ----------

Hi hliao, rafael,

This is a continuation of r167064 (
http://llvm.org/viewvc/llvm-project?view=revision&revision=167064 ).
That patch started to fix PR14221 (
http://llvm.org/bugs/show_bug.cgi?id=14221 ), but was not completed, so we
have a chunk of extra code that is preventing further improvements.

If everything looks ok here, I can submit a follow-on patch to fix sqrtsd
(double-precision square root).

I'm not locating a Phab address for Manman Ren, so I'll send a follow-on
mail.

http://reviews.llvm.org/D6330

Files:
  lib/Target/X86/X86InstrSSE.td
  test/CodeGen/X86/sse_partial_update.ll
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20141119/ad36baf0/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D6330.16406.patch
Type: text/x-patch
Size: 6429 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20141119/ad36baf0/attachment.bin>


More information about the llvm-commits mailing list