[PATCH] D153752: [InstSimplify] Fold icmp comparing GEPs with local allocation

[hanbum via Phabricator via llvm-commits]

ParkHanbum marked 2 inline comments as done.
ParkHanbum added a comment.

there was such a problem. I didn't know!! thanks for letting me.



================
Comment at: llvm/lib/Analysis/InstructionSimplify.cpp:2798-2799
+          getObjectSize(RHS, RHSSize, DL, TLI, Opts)) {
+        APInt dist;
+        dist = LHSOffset - RHSOffset;
+        dist = dist.abs();
----------------
nikic wrote:
> Variable names in LLVM start with an uppercase letter.
shoud I have to declare and assign Dist at the same time?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D153752/new/

https://reviews.llvm.org/D153752