[PATCH] D13065: [x86] replace integer 'and' ops with packed SSE FP 'and' ops when operating on FP scalars

Sanjay Patel via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 22 14:46:57 PDT 2015


spatel added inline comments.

================
Comment at: lib/Target/X86/X86ISelLowering.cpp:24453
@@ +24452,3 @@
+    // unnecessary moves from SSE to integer registers.
+    // TODO: Split this into a helper function, so it can also be used with
+    //       or/xor combining.
----------------
ab wrote:
> TODO -> FIXME ?
Sure - follow-up patch coming soon. :)
Thanks for the prompt review!


http://reviews.llvm.org/D13065





More information about the llvm-commits mailing list