From a51418c1cab874e628a110bf37a3ecfdebf05c7d Mon Sep 17 00:00:00 2001 From: Justin Holewinski Date: Fri, 11 Oct 2013 12:39:39 +0000 Subject: [PATCH] [NVPTX] Switch from StrongPHIElimination to PHIElimination in NVPTXTargetMachine, and add some missing optimization passes to addOptimizedRegAlloc Fixes PR17529 llvm-svn: 192445 --- llvm/lib/Target/NVPTX/NVPTXTargetMachine.cpp | 24 +++++++++++-- llvm/test/CodeGen/NVPTX/pr17529.ll | 38 ++++++++++++++++++++ 2 files changed, 60 insertions(+), 2 deletions(-) create mode 100644 llvm/test/CodeGen/NVPTX/pr17529.ll diff --git a/llvm/lib/Target/NVPTX/NVPTXTargetMachine.cpp b/llvm/lib/Target/NVPTX/NVPTXTargetMachine.cpp index 72afe8d6ea92..c3196552f2af 100644 --- a/llvm/lib/Target/NVPTX/NVPTXTargetMachine.cpp +++ b/llvm/lib/Target/NVPTX/NVPTXTargetMachine.cpp @@ -154,10 +154,30 @@ FunctionPass *NVPTXPassConfig::createTargetRegisterAllocator(bool) { void NVPTXPassConfig::addFastRegAlloc(FunctionPass *RegAllocPass) { assert(!RegAllocPass && "NVPTX uses no regalloc!"); - addPass(&StrongPHIEliminationID); + addPass(&PHIEliminationID); + addPass(&TwoAddressInstructionPassID); } void NVPTXPassConfig::addOptimizedRegAlloc(FunctionPass *RegAllocPass) { assert(!RegAllocPass && "NVPTX uses no regalloc!"); - addPass(&StrongPHIEliminationID); + + addPass(&ProcessImplicitDefsID); + addPass(&LiveVariablesID); + addPass(&MachineLoopInfoID); + addPass(&PHIEliminationID); + + addPass(&TwoAddressInstructionPassID); + addPass(&RegisterCoalescerID); + + // PreRA instruction scheduling. + if (addPass(&MachineSchedulerID)) + printAndVerify("After Machine Scheduling"); + + + addPass(&StackSlotColoringID); + + // FIXME: Needs physical registers + //addPass(&PostRAMachineLICMID); + + printAndVerify("After StackSlotColoring"); } diff --git a/llvm/test/CodeGen/NVPTX/pr17529.ll b/llvm/test/CodeGen/NVPTX/pr17529.ll new file mode 100644 index 000000000000..a16214225674 --- /dev/null +++ b/llvm/test/CodeGen/NVPTX/pr17529.ll @@ -0,0 +1,38 @@ +; RUN: llc < %s -march=nvptx -mcpu=sm_20 | FileCheck %s + +target datalayout = "e-p:64:64:64-i1:8:8-i8:8:8-i16:16:16-i32:32:32-i64:64:64-f32:32:32-f64:64:64-v16:16:16-v32:32:32-v64:64:64-v128:128:128-n16:32:64" +target triple = "nvptx64-nvidia-cuda" + +; Function Attrs: nounwind +; CHECK: .func kernelgen_memcpy +define ptx_device void @kernelgen_memcpy(i8* nocapture %dst) #0 { +entry: + br i1 undef, label %for.end, label %vector.body + +vector.body: ; preds = %vector.body, %entry + %index = phi i64 [ %index.next, %vector.body ], [ 0, %entry ] + %scevgep9 = getelementptr i8* %dst, i64 %index + %scevgep910 = bitcast i8* %scevgep9 to <4 x i8>* + store <4 x i8> undef, <4 x i8>* %scevgep910, align 1 + %index.next = add i64 %index, 4 + %0 = icmp eq i64 undef, %index.next + br i1 %0, label %middle.block, label %vector.body + +middle.block: ; preds = %vector.body + br i1 undef, label %for.end, label %for.body.preheader1 + +for.body.preheader1: ; preds = %middle.block + %scevgep2 = getelementptr i8* %dst, i64 0 + br label %for.body + +for.body: ; preds = %for.body, %for.body.preheader1 + %lsr.iv3 = phi i8* [ %scevgep2, %for.body.preheader1 ], [ %scevgep4, %for.body ] + store i8 undef, i8* %lsr.iv3, align 1 + %scevgep4 = getelementptr i8* %lsr.iv3, i64 1 + br label %for.body + +for.end: ; preds = %middle.block, %entry + ret void +} + +attributes #0 = { nounwind "less-precise-fpmad"="false" "no-frame-pointer-elim"="false" "no-infs-fp-math"="false" "no-nans-fp-math"="false" "no-realign-stack" "stack-protector-buffer-size"="8" "unsafe-fp-math"="false" "use-soft-float"="false" }