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.

Diff Detail

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.