[PATCH] D65360: [AArch64][GlobalISel] Eliminate redundant G_ZEXT when the source is implicitly zext-loaded

Amara Emerson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 26 19:54:24 PDT 2019


aemerson added a comment.

In D65360#1603454 <https://reviews.llvm.org/D65360#1603454>, @arsenm wrote:

> Shouldn’t the combiner have already gotten his?


It does catch these cases, but sometimes these patterns arise later after the combiner due to some other optimization or DCE. By then the combiner has already run so we don’t merge these into G_ZEXTLOAD.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65360





More information about the llvm-commits mailing list