PTAL - I ran out of time this week to make substantive progress on this, but I took a pass through the comments and cleaned up. When I next have time, I'll work on finishing the dev work on it to get everything working, then integrate it into the existing CMAKE flags and checkpoint to see where we are.
Address comments and rebase.
Hi @ggeorgakoudis, please try something along these lines:
I'd like to leave the select i1 -> and/or folding in InstCombine as a single point where such transformation happens. This patch is a step to it.
I think the only test that would make sense is a FileCheck test. Of course, that's the only kind of test I've written, so I might be deluding myself.
anymore comments or LGTM?
This change looks good to me.
Small refactor: Change recursive DFS to iterative DFS when insert tilerelease.
Address craig.topper's comment.
Undo some accidental comment formatting
I believe D94889 has fixed the issue. Were you using the trunk?