[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