[llvm] r290133 - [TargetInstrInfo] replace redundant expression in getMemOpBaseRegImmOfs

[Michael LeMay via llvm-commits]

Author: mdlemay
Date: Mon Dec 19 15:02:41 2016
New Revision: 290133

URL: http://llvm.org/viewvc/llvm-project?rev=290133&view=rev
Log:
[TargetInstrInfo] replace redundant expression in getMemOpBaseRegImmOfs

Summary:
The expression for computing the return value of getMemOpBaseRegImmOfs has only
one possible value. The other value would result in a return earlier in the
function. This patch replaces the expression with its only possible value.

Reviewers: sanjoy

Subscribers: llvm-commits

Differential Revision: https://reviews.llvm.org/D27437

Modified:
    llvm/trunk/lib/Target/X86/X86InstrInfo.cpp

Modified: llvm/trunk/lib/Target/X86/X86InstrInfo.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86InstrInfo.cpp?rev=290133&r1=290132&r2=290133&view=diff
==============================================================================
--- llvm/trunk/lib/Target/X86/X86InstrInfo.cpp (original)
+++ llvm/trunk/lib/Target/X86/X86InstrInfo.cpp Mon Dec 19 15:02:41 2016
@@ -5969,8 +5969,7 @@ bool X86InstrInfo::getMemOpBaseRegImmOfs
 
   Offset = DispMO.getImm();
 
-  return MemOp.getOperand(MemRefBegin + X86::AddrIndexReg).getReg() ==
-         X86::NoRegister;
+  return true;
 }
 
 static unsigned getStoreRegOpcode(unsigned SrcReg,