[PATCH] D55232: [SCEV] Suppress hoisting insertion point of binops when unsafe

Warren Ristow via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 7 19:51:58 PDT 2019


wristow marked 4 inline comments as done.
wristow added inline comments.


================
Comment at: test/Transforms/LoopVectorize/pr38697.ll:228
+; right-shift ('lshr') rather than an explicit division.
+; loop body.
+; INDVARCHECK-LABEL: DivBy16Hoist
----------------
sanjoy wrote:
> Can you please make these CHECK lines a bit more specific?  Maybe using `update_test_checks.py`?
I've addressed this to some extent, but edited it down significantly.  Let me know if you'd prefer the larger set of tests.


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

https://reviews.llvm.org/D55232





More information about the llvm-commits mailing list