[PATCH] D130994: [DAGCombiner] Hoist funnel shifts from logic operation.

Fri Aug 5 07:48:30 PDT 2022

spatel added a comment.

In D130994#3702138 <https://reviews.llvm.org/D130994#3702138>, @fzhinkin wrote:

> In D130994#3700572 <https://reviews.llvm.org/D130994#3700572>, @spatel wrote:
>
>> Please pre-commit the new tests with baseline results, so we'll just show the diffs in this patch.
>
> Unfortunately, I don't have a permission to commit. Here's a patch adding tests with checks generated using llc build from the main branch: F24025151: D130994_precommit_tests.patch <https://reviews.llvm.org/F24025151>.
>
> I'll appreciate if you can help me with committing it.

Sure: 249a7ed75072 <https://reviews.llvm.org/rG249a7ed750729b79341fb22003a0f4537883d928>

>> Add an alive2 proof link for at least one of the patterns to the patch description, so we can show correctness.
>
> Done, thanks!

Great - please update the patch here Phab with the test changes.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130994