[PATCH] D94014: [InstCombine] reduce icmp(ashr X, C1), C2 to sign-bit test

[Sanjay Patel via Phabricator via llvm-commits]

spatel marked an inline comment as done.
spatel added inline comments.


================
Comment at: llvm/test/Transforms/InstCombine/icmp-shr.ll:512
+
+: negative test, but different transform possible
+
----------------
lebedev.ri wrote:
> It's a `:` (colon), not a `;` (semicolon), does this test parse?
Oops - I pasted in the test comments after running `ninja check`, so missed the typo.


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

https://reviews.llvm.org/D94014