[PATCH] [X86] Break false dependencies before partial register updates when the source operand is in memory

Simon Pilgrim llvm-dev at redking.me.uk
Sun Dec 14 02:48:07 PST 2014


Please can you double check that cvtsd2ss test and if possible add tests for what other partial ops you can?

Also - should your test case be under sse-domains.ll or one of the break-*-dep.ll test files?

Unless you are wanting your reviewers to confirm I'm happy to approve this after that.

Cheers, Simon.


================
Comment at: test/CodeGen/X86/break-sse-dep.ll:18
@@ -17,3 +17,3 @@
 ; CHECK-LABEL: t2:
 ; CHECK: cvtsd2ss ([[A0]]), %xmm0
   %0 = load double* %x, align 8
----------------
Should cvtsd2ss be folded?

http://reviews.llvm.org/D6620

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list