[PATCH] D154717: [LV] Check if ops can safely be truncated in computeMinimumValueSizes.

[Nikita Popov via Phabricator via llvm-commits]

nikic accepted this revision.
nikic added a comment.
This revision is now accepted and ready to land.

LG



================
Comment at: llvm/lib/Analysis/VectorUtils.cpp:676
+            auto *CI = dyn_cast<ConstantInt>(U);
+            // For constants shit amounts, check if the shift would result in
+            // poison.
----------------
shit -> shift :P


================
Comment at: llvm/test/Transforms/LoopVectorize/trunc-shifts.ll:6
 
 ; FIXME: Currently the operands of %l are incorrectly truncated.
 define void @test_pr47927_lshr_const_shift_ops(ptr %dst, i32 %f) {
----------------
Drop FIXME


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D154717