[PATCH] D34335: Fix invalid ptrtoint in InstCombine

Keno Fischer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 29 12:33:15 PDT 2017


loladiro added inline comments.


================
Comment at: lib/Analysis/Loads.cpp:373
 
+        if (Ptr->getType()->isPointerTy()) {
+          PointerType *Ty1 = cast<PointerType>(Ptr->getType());
----------------
I'm a bit confused by this. Isn't `Ptr` the thing we're loading from? Why are we comparing it against `LI->getType()` shouldn't we be looking at `AccessTy`?


https://reviews.llvm.org/D34335





More information about the llvm-commits mailing list