[all-commits] [llvm/llvm-project] e1cebe: [InstCombine] add tests for icmp-of-and-of-select-...
Sanjay Patel via All-commits
all-commits at lists.llvm.org
Mon Jan 30 13:19:15 PST 2023
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: e1cebe9ed12a46282fb6cc9fa9e172b04e5474ef
https://github.com/llvm/llvm-project/commit/e1cebe9ed12a46282fb6cc9fa9e172b04e5474ef
Author: Sanjay Patel <spatel at rotateright.com>
Date: 2023-01-30 (Mon, 30 Jan 2023)
Changed paths:
M llvm/test/Transforms/InstCombine/icmp-select.ll
Log Message:
-----------
[InstCombine] add tests for icmp-of-and-of-select-of-constants; NFC
This overlaps with the tests proposed in D134064
and providing coverage for D133919, but it is
both limited in scope and more comprehensive
because there are many potential ways to transform
these kinds of patterns.
Commit: 98855059674cf1b6b415f8c543f2a923771fed27
https://github.com/llvm/llvm-project/commit/98855059674cf1b6b415f8c543f2a923771fed27
Author: Sanjay Patel <spatel at rotateright.com>
Date: 2023-01-30 (Mon, 30 Jan 2023)
Changed paths:
M llvm/lib/Transforms/InstCombine/InstCombineCompares.cpp
M llvm/test/Transforms/InstCombine/icmp-select.ll
Log Message:
-----------
[InstCombine] reduce icmp_eq0-of-and-of-select-of-constants
This is the most basic patch to handle fixing issue #57666.
D133919 proposes to handle much more than this in a single patch,
but I've used 10 regression tests just to make sure this part is
doing what I expected and nothing more, and it already shows even
more potential TODO items.
The more general proofs from D133919 are correct, but I want to
enable this in smaller steps to reduce risk:
https://alive2.llvm.org/ce/z/RrVEyX
Differential Revision: https://reviews.llvm.org/D142847
Compare: https://github.com/llvm/llvm-project/compare/c9986f826398...98855059674c
More information about the All-commits
mailing list