This is an archive of the discontinued LLVM Phabricator instance.

[AArch64][SVE] Gather loads: pass 32 bit unpacked offsets as nxv2i32
ClosedPublic

Authored by andwar on Dec 19 2019, 12:55 PM.

Details

Summary

Currently 32 bit unpacked offsets are passed as nxv2i64. However, as
pointed out in https://reviews.llvm.org/D71074, using nxv2i32 instead
would improve consistency with:

  • how other arguments are treated
  • how scatter stores are implemented

This patch makes sure that 32 bit unpacked offsets are passes as nxv2i32
instead of nxv2i64.

Event Timeline

andwar created this revision.Dec 19 2019, 12:55 PM
Herald added a project: Restricted Project. · View Herald Transcript
This revision is now accepted and ready to land.Dec 19 2019, 2:17 PM
This revision was automatically updated to reflect the committed changes.