[PATCH] D108772: [LSR] Make sure that Factor fits into Base type

Quentin Colombet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 17 09:21:44 PDT 2021


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

One last comment, otherwise LGTM.



================
Comment at: llvm/test/Transforms/LoopStrengthReduce/scaling-factor-incompat-type.ll:1
-; Check that it doesn't crash
+; Check that it doesn't crash, see pr42770
 ; REQUIRES: asserts
----------------
Could you add a more descriptive comment on what this is testing and how? (I.e., the characteristics of the test and what we should be observing when the optimization does the right thing.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108772



More information about the llvm-commits mailing list