[PATCH] D81993: [AArch64][GlobalISel] Add artifact combine for sext(trunc(sextload)) -> trunc/copy

Amara Emerson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 17 10:13:27 PDT 2020


aemerson added a comment.

In D81993#2097971 <https://reviews.llvm.org/D81993#2097971>, @arsenm wrote:

> This doesn't seem necessary for correctness? I think we do need some optimizing combines in the legalizer itself, but I assumed we would start putting those in a distinct place


True, I suppose we need to be stricter about these combines. I'll move this into the post legalizer combiner.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81993





More information about the llvm-commits mailing list