From 46abcb236bb4929a08d4905bf2169ec592202454 Mon Sep 17 00:00:00 2001 From: Krzysztof Parzyszek <kparzysz@codeaurora.org> Date: Fri, 30 Mar 2018 15:09:05 +0000 Subject: [PATCH] [Hexagon] Fix printing :mem_noshuf on compiler-generated packets llvm-svn: 328869 --- llvm/lib/Target/Hexagon/HexagonAsmPrinter.cpp | 9 +++- llvm/test/CodeGen/Hexagon/swp-check-offset.ll | 46 +++++++++++++++++++ 2 files changed, 53 insertions(+), 2 deletions(-) create mode 100644 llvm/test/CodeGen/Hexagon/swp-check-offset.ll diff --git a/llvm/lib/Target/Hexagon/HexagonAsmPrinter.cpp b/llvm/lib/Target/Hexagon/HexagonAsmPrinter.cpp index 3a1821bf265f..5be2cc9145ed 100644 --- a/llvm/lib/Target/Hexagon/HexagonAsmPrinter.cpp +++ b/llvm/lib/Target/Hexagon/HexagonAsmPrinter.cpp @@ -755,9 +755,14 @@ void HexagonAsmPrinter::EmitInstruction(const MachineInstr *MI) { for (++MII; MII != MBB->instr_end() && MII->isInsideBundle(); ++MII) if (!MII->isDebugValue() && !MII->isImplicitDef()) HexagonLowerToMC(MCII, &*MII, MCB, *this); - } - else + } else { HexagonLowerToMC(MCII, MI, MCB, *this); + } + + const MachineFunction &MF = *MI->getParent()->getParent(); + const auto &HII = *MF.getSubtarget<HexagonSubtarget>().getInstrInfo(); + if (MI->isBundle() && HII.getBundleNoShuf(*MI)) + HexagonMCInstrInfo::setMemReorderDisabled(MCB); bool Ok = HexagonMCInstrInfo::canonicalizePacket( MCII, *Subtarget, OutStreamer->getContext(), MCB, nullptr); diff --git a/llvm/test/CodeGen/Hexagon/swp-check-offset.ll b/llvm/test/CodeGen/Hexagon/swp-check-offset.ll new file mode 100644 index 000000000000..2654aaabe743 --- /dev/null +++ b/llvm/test/CodeGen/Hexagon/swp-check-offset.ll @@ -0,0 +1,46 @@ +; RUN: llc -march=hexagon -mcpu=hexagonv5 -enable-pipeliner < %s | FileCheck %s +; RUN: llc -march=hexagon -mcpu=hexagonv62 -enable-pipeliner < %s | FileCheck --check-prefix=CHECK-V62 %s +; RUN: llc -march=hexagon -mcpu=hexagonv65 -enable-pipeliner < %s | FileCheck --check-prefix=CHECK-V65 %s + +; +; Make sure we pipeline the loop and that we generate the correct +; base+offset values for the loads. + +; CHECK: loop0(.LBB0_[[LOOP:.]], +; CHECK: .LBB0_[[LOOP]]: +; CHECK: r{{[0-9]+}} = memw([[REG1:(r[0-9]+)]]+#{{[0,4]}}) +; CHECK: r{{[0-9]+}} = memw([[REG1]]++#4) +; CHECK: }{{[ \t]*}}:endloop +; CHECK-V62-NOT: }{{[ \t]*}}:mem_noshuf +; CHECK-V65: }{{[ \t]*}}:mem_noshuf + +; Function Attrs: nounwind +define void @f0() #0 { +b0: + br i1 undef, label %b1, label %b4 + +b1: ; preds = %b1, %b0 + %v0 = phi i32 [ %v7, %b1 ], [ 0, %b0 ] + %v1 = getelementptr inbounds i8*, i8** undef, i32 %v0 + %v2 = load i8*, i8** %v1, align 4 + %v3 = bitcast i8* %v2 to i32* + store i32 0, i32* %v3, align 4 + %v4 = load i8*, i8** %v1, align 4 + %v5 = getelementptr inbounds i8, i8* %v4, i32 8 + %v6 = bitcast i8* %v5 to i32* + store i32 0, i32* %v6, align 4 + %v7 = add nsw i32 %v0, 1 + %v8 = icmp eq i32 %v7, 2 + br i1 %v8, label %b2, label %b1 + +b2: ; preds = %b1 + br i1 undef, label %b3, label %b4 + +b3: ; preds = %b2 + unreachable + +b4: ; preds = %b2, %b0 + unreachable +} + +attributes #0 = { nounwind }