Subzero: Aggressive LEA

src/IceTargetLoweringX86BaseImpl.h

 //===- subzero/src/IceTargetLoweringX86BaseImpl.h - x86 lowering -*- C++ -*-==//
 //
 //                        The Subzero Code Generator
 //
 // This file is distributed under the University of Illinois Open Source
 // License. See LICENSE.TXT for details.
 //
 //===----------------------------------------------------------------------===//
 ///
 /// \file
@@ skipping 2165 matching lines @@
       break;
     }
     return;
   }
   Variable *T_edx = nullptr;
   Variable *T = nullptr;
   switch (Instr->getOp()) {
   case InstArithmetic::_num:
     llvm_unreachable("Unknown arithmetic operator");
     break;
-  case InstArithmetic::Add:
-    _mov(T, Src0);
-    _add(T, Src1);
-    _mov(Dest, T);
-    break;
+  case InstArithmetic::Add: {
+    bool ConvertedToLea = false;
+    if (getFlags().getAggressiveLea() && Func->getOptLevel() == Opt_2 &&

[Jim Stichnoth, 2016/07/19 21:29:08]

Do you really need OptLevel=2?  or is getAggressiveLea() sufficient?

[manasijm, 2016/07/28 18:01:39]

Done.

+        (Ty == IceType_i32 || (Ty == IceType_i64 && Traits::Is64Bit))) {
+      Constant *Const = llvm::dyn_cast<Constant>(Instr->getSrc(1));
+      if (Const != nullptr && (llvm::isa<ConstantInteger32>(Const) ||
+                               llvm::isa<ConstantRelocatable>(Const))) {
+        auto Var = legalize(Src0, Legal_Reg);

[Jim Stichnoth, 2016/07/19 21:29:08]

auto *

Also, in this case you can use legalizeToReg() for a simpler and more consistent
style.

[manasijm, 2016/07/28 18:01:39]

Done.

+        auto Mem = Traits::X86OperandMem::create(
+            Func, IceType_void, llvm::cast<Variable>(Var), Const);
+        T = makeReg(Ty);
+        _lea(T, _sandbox_mem_reference(Mem));
+        _mov(Dest, T);
+
+        ConvertedToLea = true;
+      } /*else { // Both are Variables

[Jim Stichnoth, 2016/07/19 21:29:09]

Why is the rest of this commented out?

[manasijm, 2016/07/19 21:54:59]

Enabling that increases code size.
And doesn't seem to be helpful in general.
The commented out portion would have converted x = y + z to a lea and resulted
in more forced register legalizations.

[Jim Stichnoth, 2016/07/19 22:30:29]

OK.  Either remove the code, or leave it in but disabled and commented.  E.g.:

  } else if (false) {
    // TODO: Find a good strategy for enabling this.
  }

That way, the disabled code still gets compiled and is less likely to rot.

[manasijm, 2016/07/28 18:01:39]

Done.

+        auto Var0 = legalize(Src0, Legal_Reg);
+        auto Var1 = legalize(Src1, Legal_Reg);
+        auto Mem = Traits::X86OperandMem::create(
+            Func, IceType_void, llvm::cast<Variable>(Var0), nullptr,

[Jim Stichnoth, 2016/07/19 21:29:08]

If you use legalizeToReg() as above, you won't need the llvm::cast<Variable>()
here.

[manasijm, 2016/07/28 18:01:39]

Done.

+      llvm::cast<Variable>(Var1));
+
+        T = makeReg(Ty);
+        _lea(T, _sandbox_mem_reference(Mem));
+        _mov(Dest, T);
+
+        ConvertedToLea = true;
+      }*/
+    }
+    if (!ConvertedToLea) {
+      _mov(T, Src0);
+      _add(T, Src1);
+      _mov(Dest, T);
+    }
+  } break;
   case InstArithmetic::And:
     _mov(T, Src0);
     _and(T, Src1);
     _mov(Dest, T);
     break;
   case InstArithmetic::Or:
     _mov(T, Src0);
     _or(T, Src1);
     _mov(Dest, T);
     break;
@@ skipping 5833 matching lines @@
         emitGlobal(*Var, SectionSuffix);
       }
     }
   } break;
   }
 }
 } // end of namespace X86NAMESPACE
 } // end of namespace Ice
 
 #endif // SUBZERO_SRC_ICETARGETLOWERINGX86BASEIMPL_H