[PATCH] D135700: [DAGCombine] Simplify (truncate (build_pair x, y)) -> (truncate x) or x
Krzysztof Parzyszek via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Oct 31 10:44:14 PDT 2022
kparzysz added a comment.
In D135700#3885112 <https://reviews.llvm.org/D135700#3885112>, @RKSimon wrote:
> What do you want to do next - merge your local branch Hexagon patch into trunk?
Sorry, missed this comment. The Hexagon-specific code is already merged. As for this, I was planning to fix the x86 codegen to get this case, but I haven't had time to do it yet. I believe this only applies to extending loads, where the consumer accesses the extension bits, and only when the consumer can be modified to access the memory directly. Only existing patterns/selections would need to be updated, because without such explicit change from using register to memory this patch wouldn't have any effect on the code size.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D135700/new/
https://reviews.llvm.org/D135700
More information about the llvm-commits
mailing list