[llvm] [AArch64][SVE] Don't require 16-byte aligned SVE loads/stores with +strict-align (PR #119732)
Paul Walker via llvm-commits
llvm-commits at lists.llvm.org
Fri Dec 13 06:27:00 PST 2024
================
@@ -2569,6 +2569,17 @@ MVT AArch64TargetLowering::getScalarShiftAmountTy(const DataLayout &DL,
bool AArch64TargetLowering::allowsMisalignedMemoryAccesses(
EVT VT, unsigned AddrSpace, Align Alignment, MachineMemOperand::Flags Flags,
unsigned *Fast) const {
+
+ // Allow SVE loads/stores where the alignment >= the size of the element type,
+ // even with +strict-align. The SVE loads/stores do not require memory to be
+ // aligned more than the element type even without unaligned accesses.
+ // Without this, already aligned loads and stores are forced to have 16-byte
+ // alignment, which is unnecessary and fails to build as
+ // TLI.expandUnalignedLoad() and TLI.expandUnalignedStore() don't yet support
+ // scalable vectors.
----------------
paulwalker-arm wrote:
Great news. Thanks for the update.
https://github.com/llvm/llvm-project/pull/119732
More information about the llvm-commits
mailing list