[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