[PATCH] D13717: [IndVars] Have `cloneArithmeticIVUser` guess better

[Sanjoy Das via llvm-commits]

sanjoy added inline comments.

================
Comment at: lib/Transforms/Scalar/IndVarSimplify.cpp:1020
@@ +1019,3 @@
+
+    case Instruction::Add:
+      WideOp = SE->getAddExpr(WideLHS, WideRHS);
----------------
reames wrote:
> Isn't there something along the lines of SE->getBinOp?
Coud not find any such thing.

================
Comment at: lib/Transforms/Scalar/IndVarSimplify.cpp:1037
@@ +1036,3 @@
+
+    return WideOp == WideAR;
+  };
----------------
reames wrote:
> Comment?  Where'd WideAR come from?
Added a comment explaining what's going on.


http://reviews.llvm.org/D13717