commit a870bc5495b20a830ebd8342b49ef148bbff72dd
author Haitao Feng <haitao.feng@intel.com> Tue Sep 09 15:52:34 2014 +0800
committer Haitao Feng <haitao.feng@intel.com> Thu Sep 11 19:55:18 2014 +0800
tree 32831cd981d349ed601bccd26c1f0d73742bce34
parent 33c1693244996102c3aa37d2f54e02220d77f62b

ART: Address three issues with x86 assembler before enabling load store elimination.

1) Remove the IS_LOAD attribute from LEA instructions.
2) Change the attribute of fp stack instructions from IS_UNARY_OP to IS_BINARY_OP as
   operands[1] will be used to compute GetInstructionOffset.
3) Add IS_MOVE attribute for general register move instructions.

Change-Id: I7054df47956f2acecf579ff7acfde385fd8ac194
Signed-off-by: Haitao Feng <haitao.feng@intel.com>

compiler/dex/quick/mir_to_lir-inl.h

diff --git a/compiler/dex/quick/mir_to_lir-inl.h b/compiler/dex/quick/mir_to_lir-inl.h
index 2e4e292..22588f3 100644
--- a/compiler/dex/quick/mir_to_lir-inl.h
+++ b/compiler/dex/quick/mir_to_lir-inl.h
@@ -97,7 +97,7 @@
 }
 
 inline LIR* Mir2Lir::NewLIR2NoDest(int opcode, int src, int info) {
-  DCHECK(IsPseudoLirOp(opcode) || (GetTargetInstFlags(opcode) & IS_UNARY_OP))
+  DCHECK(IsPseudoLirOp(opcode) || (GetTargetInstFlags(opcode) & IS_BINARY_OP))
       << GetTargetInstName(opcode) << " " << opcode << " "
       << PrettyMethod(cu_->method_idx, *cu_->dex_file) << " "
       << current_dalvik_offset_;