[PATCH] D154938: [InstCombine] Test cases for D154937

[Dhruv Chawla via Phabricator via llvm-commits]

0xdc03 marked an inline comment as done.
0xdc03 added inline comments.


================
Comment at: llvm/test/Transforms/InstCombine/and-or-icmp-const-icmp.ll:503
+  ret <2 x i1> %and
+}
+
----------------
nikic wrote:
> I'd personally save a lot of these tests -- generally we want to see one test of each "kind", but not every possible combination. So a test for eq, ne, vectors and mismatched constants each, rather than all 8 combinations of eq/ne, scalar/vector and matching/mismatching for example.
That's a great idea, I have halved the test count by interpolating the eq and ne tests :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D154938