[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