[llvm] r350076 - [WebAssembly] Delete an unnecessary line in RegStackify

Heejin Ahn via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 26 14:33:35 PST 2018


Author: aheejin
Date: Wed Dec 26 14:33:35 2018
New Revision: 350076

URL: http://llvm.org/viewvc/llvm-project?rev=350076&view=rev
Log:
[WebAssembly] Delete an unnecessary line in RegStackify

`OneUseInst` is set outside of the loop before and `OneUse` does not
change throughout the loop, so this line is not necessary.

Modified:
    llvm/trunk/lib/Target/WebAssembly/WebAssemblyRegStackify.cpp

Modified: llvm/trunk/lib/Target/WebAssembly/WebAssemblyRegStackify.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/WebAssembly/WebAssemblyRegStackify.cpp?rev=350076&r1=350075&r2=350076&view=diff
==============================================================================
--- llvm/trunk/lib/Target/WebAssembly/WebAssemblyRegStackify.cpp (original)
+++ llvm/trunk/lib/Target/WebAssembly/WebAssemblyRegStackify.cpp Wed Dec 26 14:33:35 2018
@@ -414,7 +414,6 @@ static bool OneUseDominatesOtherUses(uns
     if (UseVNI != OneUseVNI)
       continue;
 
-    const MachineInstr *OneUseInst = OneUse.getParent();
     if (UseInst == OneUseInst) {
       // Another use in the same instruction. We need to ensure that the one
       // selected use happens "before" it.




More information about the llvm-commits mailing list