This is an archive of the discontinued LLVM Phabricator instance.

[llvm][CodeGen] Rename SVE gather prefetch intrinsics. [NFC]
ClosedPublic

Authored by fpetrogalli on Apr 9 2020, 4:20 PM.

Details

Summary

The renaming is necessary to make the naming scheme uniform with other
gather/scatter load/stores SVE intrinsics.

The naming of variables and functions have been adapted to make it
explicit whether we are dealing with a scalar offset (which is
unscaled) or an index (which is scaled according to the data type of
the lanes of the vector).

Diff Detail

Event Timeline

fpetrogalli created this revision.Apr 9 2020, 4:20 PM
andwar accepted this revision.Apr 14 2020, 9:58 AM

LGTM, thanks for the improvements!

llvm/lib/Target/AArch64/AArch64ISelLowering.cpp
13023

UPDATEME: aarch64_sve_prf<T>_gather_scaled_uxtw

llvm/test/CodeGen/AArch64/sve-intrinsics-gather-prefetches-scalar-base-vector-indexes.ll
22

DELETEME

37

Can I be followed by an empty line, please?

This revision is now accepted and ready to land.Apr 14 2020, 9:58 AM
This revision was automatically updated to reflect the committed changes.
llvm/test/CodeGen/AArch64/sve-intrinsics-gather-prefetches-scalar-base-vector-indexes.ll