[PATCH] D22580: Fix bug in clearance calculation

Marina Yatsina via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 21 05:44:47 PDT 2016


This revision was automatically updated to reflect the committed changes.
Closed by commit rL276266: ExecutionDepsFix - Fix bug in clearance calculation (authored by myatsina).

Changed prior to commit:
  https://reviews.llvm.org/D22580?vs=64727&id=64863#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D22580

Files:
  llvm/trunk/lib/CodeGen/ExecutionDepsFix.cpp
  llvm/trunk/test/CodeGen/X86/break-false-dep.ll

Index: llvm/trunk/lib/CodeGen/ExecutionDepsFix.cpp
===================================================================
--- llvm/trunk/lib/CodeGen/ExecutionDepsFix.cpp
+++ llvm/trunk/lib/CodeGen/ExecutionDepsFix.cpp
@@ -520,8 +520,6 @@
     MachineOperand &MO = MI->getOperand(i);
     if (!MO.isReg())
       continue;
-    if (MO.isImplicit())
-      break;
     if (MO.isUse())
       continue;
     for (int rx : regIndices(MO.getReg())) {
Index: llvm/trunk/test/CodeGen/X86/break-false-dep.ll
===================================================================
--- llvm/trunk/test/CodeGen/X86/break-false-dep.ll
+++ llvm/trunk/test/CodeGen/X86/break-false-dep.ll
@@ -199,3 +199,13 @@
 ;AVX-NEXT: vmulsd {{.*}}, [[XMM0]], [[XMM0]]
 ;AVX-NEXT: vmovsd [[XMM0]],
 }
+
+define double @inlineasmdep(i64 %arg) {
+top:
+  tail call void asm sideeffect "", "~{xmm0},~{dirflag},~{fpsr},~{flags}"()
+  %tmp1 = sitofp i64 %arg to double
+  ret double %tmp1
+;AVX-LABEL:@inlineasmdep
+;AVX: vxorps  [[XMM0:%xmm[0-9]+]], [[XMM0]], [[XMM0]]
+;AVX-NEXT: vcvtsi2sdq {{.*}}, [[XMM0]], {{%xmm[0-9]+}}
+}


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D22580.64863.patch
Type: text/x-patch
Size: 1089 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160721/6dd3381f/attachment.bin>


More information about the llvm-commits mailing list