[PATCH] D113835: [CVP] Remove ashr of -1 or 0

[Philip Reames via Phabricator via llvm-commits]

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

LGTM with required changes made.



================
Comment at: llvm/test/Transforms/CorrelatedValuePropagation/ashr.ll:121
+; CHECK-LABEL: @test7
+define i32 @test7(i32 %f, i32 %g) {
+entry:
----------------
Please add a couple of tests for edge cases.  (e.g. ashr -2, X, ashr 1, X)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113835