[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