[PATCH] D116942: GlobalISel: Fix legalization error where CSE leaves behind dead defs
Petar Avramovic via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 10 08:34:35 PST 2022
Petar.Avramovic accepted this revision.
Petar.Avramovic added a comment.
This revision is now accepted and ready to land.
LGTM.
> The G_UNMERGE_VALUES of the G_SEXT of the G_BUILD_VECTOR would introduce a G_SEXT for each of the scalars.
Is there a reason why we don't have simpler pattern where `conversion opcode` of `merge like opcode` is not used by G_UNMERGE_VALUES (in this case only G_SEXT of the G_BUILD_VECTOR -> G_BUILD_VECTOR of G_SEXT for each of the scalars).
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D116942/new/
https://reviews.llvm.org/D116942
More information about the llvm-commits
mailing list