[PATCH] D129653: isInductionPHI - Add some safety checks

[Zaara Syeda via Phabricator via llvm-commits]

syzaara added inline comments.


================
Comment at: llvm/lib/Analysis/IVDescriptors.cpp:1509
+
+  if (!Phi)
+    return false;
----------------
nickdesaulniers wrote:
> fhahn wrote:
> > This should never be called with `Phi` being null.
> Sounds like a good case for Phi being a `&` (and additional `const`) rather than a non-const ptr.
This would require many changes to all the places we call isInductionPHI from. Can we add the null check for now, and create a separate PR for this change? @fhahn 


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

https://reviews.llvm.org/D129653