[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