This is an archive of the discontinued LLVM Phabricator instance.

[AArch64][GlobalISel] Add support for narrowScalar of G_ZEXT
ClosedPublic

Authored by aemerson on Aug 13 2019, 3:43 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

aemerson created this revision.Aug 13 2019, 3:43 PM
arsenm added a subscriber: arsenm.Aug 13 2019, 5:23 PM
arsenm added inline comments.
llvm/lib/CodeGen/GlobalISel/LegalizerHelper.cpp
622 ↗(On Diff #214955)

This should be able to handle cases where the result size is any multiple of the narrow size

aemerson updated this revision to Diff 215012.Aug 13 2019, 7:10 PM

Add support for any multiple of the narrow type.

This revision is now accepted and ready to land.Aug 20 2019, 3:07 PM
This revision was automatically updated to reflect the committed changes.

Hi! After running a bisect, I believe this patch is causing the assertion error in https://bugs.llvm.org/show_bug.cgi?id=43171. Could you look into this? There is a reproducer attached to the bug.

llvm/trunk/lib/Target/AArch64/AArch64LegalizerInfo.cpp