R600: Do not mergevector after a vector reg is used
authorVincent Lejeune <vljn@ovi.com>
Wed, 31 Jul 2013 19:32:12 +0000 (19:32 +0000)
committerVincent Lejeune <vljn@ovi.com>
Wed, 31 Jul 2013 19:32:12 +0000 (19:32 +0000)
If we merge vector when a vector is used, it will generate an artificial
antidependency that can prevent 2 tex/vtx instructions to use the same
clause and thus generate extra clauses that reduce performance.

There is no test case as such situation is really hard to predict.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@187516 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/R600/R600OptimizeVectorRegisters.cpp

index c47bc398d5fdedf0e2042faab4203f4077d47bd4..acacffa85d5f19eea98e63a05271e3e3ba18b668 100644 (file)
@@ -322,8 +322,17 @@ bool R600VectorRegMerger::runOnMachineFunction(MachineFunction &Fn) {
     for (MachineBasicBlock::iterator MII = MB->begin(), MIIE = MB->end();
          MII != MIIE; ++MII) {
       MachineInstr *MI = MII;
-      if (MI->getOpcode() != AMDGPU::REG_SEQUENCE)
+      if (MI->getOpcode() != AMDGPU::REG_SEQUENCE) {
+        if (TII->get(MI->getOpcode()).TSFlags & R600_InstFlag::TEX_INST) {
+          unsigned Reg = MI->getOperand(1).getReg();
+          for (MachineRegisterInfo::def_iterator It = MRI->def_begin(Reg),
+              E = MRI->def_end(); It != E; ++It) {
+            RemoveMI(&(*It));
+          }
+        }
         continue;
+      }
+
 
       RegSeqInfo RSI(*MRI, MI);