[PATCH] D16809: [JumpThreading] See through Cast Instructions

Balaram Makam via llvm-commits llvm-commits at lists.llvm.org
Sun Feb 7 16:26:25 PST 2016


bmakam added inline comments.

================
Comment at: lib/Transforms/Scalar/JumpThreading.cpp:415
@@ +414,3 @@
+    if (Value *NewV = SimplifyInstruction(I, BB->getModule()->getDataLayout()))
+      V = NewV;
+
----------------
bmakam wrote:
> You might need to RAUW NewV, 
> NewV->replaceAllUsesWith(V);
> V->eraseFromParent();
> 
I meant V->replaceAllUsesWith(NewV)


Repository:
  rL LLVM

http://reviews.llvm.org/D16809





More information about the llvm-commits mailing list