[llvm] r352817 - GlobalISel: Fix MMO creation with non-power-of-2 mem size
Matt Arsenault via llvm-commits
llvm-commits at lists.llvm.org
Thu Jan 31 15:41:23 PST 2019
Author: arsenm
Date: Thu Jan 31 15:41:23 2019
New Revision: 352817
URL: http://llvm.org/viewvc/llvm-project?rev=352817&view=rev
Log:
GlobalISel: Fix MMO creation with non-power-of-2 mem size
It should probably just be mandatory for getTgtMemIntrinsic to return
the alignment.
Modified:
llvm/trunk/lib/CodeGen/GlobalISel/IRTranslator.cpp
llvm/trunk/test/CodeGen/AArch64/GlobalISel/arm64-irtranslator.ll
Modified: llvm/trunk/lib/CodeGen/GlobalISel/IRTranslator.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/GlobalISel/IRTranslator.cpp?rev=352817&r1=352816&r2=352817&view=diff
==============================================================================
--- llvm/trunk/lib/CodeGen/GlobalISel/IRTranslator.cpp (original)
+++ llvm/trunk/lib/CodeGen/GlobalISel/IRTranslator.cpp Thu Jan 31 15:41:23 2019
@@ -1213,12 +1213,13 @@ bool IRTranslator::translateCall(const U
TargetLowering::IntrinsicInfo Info;
// TODO: Add a GlobalISel version of getTgtMemIntrinsic.
if (TLI.getTgtMemIntrinsic(Info, CI, *MF, ID)) {
- uint64_t Size = Info.memVT.getStoreSize();
- if (Info.align == 0)
- Info.align = Size;
+ unsigned Align = Info.align;
+ if (Align == 0)
+ Align = DL->getABITypeAlignment(Info.memVT.getTypeForEVT(F->getContext()));
+ uint64_t Size = Info.memVT.getStoreSize();
MIB.addMemOperand(MF->getMachineMemOperand(MachinePointerInfo(Info.ptrVal),
- Info.flags, Size, Info.align));
+ Info.flags, Size, Align));
}
return true;
Modified: llvm/trunk/test/CodeGen/AArch64/GlobalISel/arm64-irtranslator.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/AArch64/GlobalISel/arm64-irtranslator.ll?rev=352817&r1=352816&r2=352817&view=diff
==============================================================================
--- llvm/trunk/test/CodeGen/AArch64/GlobalISel/arm64-irtranslator.ll (original)
+++ llvm/trunk/test/CodeGen/AArch64/GlobalISel/arm64-irtranslator.ll Thu Jan 31 15:41:23 2019
@@ -2323,3 +2323,12 @@ define float @test_sqrt_f32(float %x) {
%y = call float @llvm.sqrt.f32(float %x)
ret float %y
}
+
+; CHECK-LABEL: name: test_llvm.aarch64.neon.ld3.v4i32.p0i32
+; CHECK: %1:_(s384) = G_INTRINSIC_W_SIDE_EFFECTS intrinsic(@llvm.aarch64.neon.ld3), %0(p0) :: (load 48 from %ir.ptr, align 64)
+define void @test_llvm.aarch64.neon.ld3.v4i32.p0i32(i32* %ptr) {
+ %arst = call { <4 x i32>, <4 x i32>, <4 x i32> } @llvm.aarch64.neon.ld3.v4i32.p0i32(i32* %ptr)
+ ret void
+}
+
+declare { <4 x i32>, <4 x i32>, <4 x i32> } @llvm.aarch64.neon.ld3.v4i32.p0i32(i32*) #3
More information about the llvm-commits
mailing list