[PATCH] D148186: ValueTracking: Handle insertelement in computeKnownFPClass

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 12 18:40:30 PDT 2023


arsenm added inline comments.


================
Comment at: llvm/lib/Analysis/ValueTracking.cpp:4549
+    if (!CIdx || CIdx->getValue().uge(NumElts)) {
+      Known.resetAll();
+      return;
----------------
I think these resets aren't necessary (and probably aren't in the computeKnownBits version either)


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

https://reviews.llvm.org/D148186



More information about the llvm-commits mailing list