[PATCH] D141653: [X86] Improve instruction ordering of constant `srl/shl` with `and` to get better and-masks
Noah Goldstein via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Mar 16 10:42:00 PDT 2023
goldstein.w.n added a comment.
In D141653#4198903 <https://reviews.llvm.org/D141653#4198903>, @RKSimon wrote:
> @goldstein.w.n What is happening with this patch? After they leave my "Ready to Review" list I tend to lose track......
When I tested on bootstrap build it caused on infinite loop. I think this approach is inherently brittle and susceptible
to that sort of bug. I haven't quite abandoned it as I think some of the shl/shr improvements can be salvaged. But
the `imm` improvedments I want to move to a new pass that runs at the way of end DAG lowering.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D141653/new/
https://reviews.llvm.org/D141653
More information about the llvm-commits
mailing list