[PATCH] D86989: [InstCombine] Fix a couple crahses with extractelement on a scalable vector.

[David Sherwood via Phabricator via llvm-commits]

david-arm added inline comments.


================
Comment at: llvm/lib/Analysis/ValueTracking.cpp:2573
+      unsigned NumElts = VecTy->getNumElements();
+      APInt DemandedVecElts = APInt::getAllOnesValue(NumElts);
+      if (CIdx && CIdx->getValue().ult(NumElts))
----------------
sdesmalen wrote:
> nit: is this how whitespace changes are rendered after the latest Phabricator update, or are these tabs?
I've seen these rendered on mine and other patches too where there are no tabs. I think it's the latest Phabricator update showing indentation.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86989