mirror of
https://src.fedoraproject.org/rpms/llvm.git
synced 2024-11-25 01:35:17 +00:00
revert upstream radeonsi breaking change.
This commit is contained in:
Dave Airlie
parent
a5fe5a1bb7
commit
97474644bf
2 changed files with 126 additions and 1 deletions
119
0001-Revert-Merging-r280589.patch
Normal file
119
0001-Revert-Merging-r280589.patch
Normal file
|
|
@ -0,0 +1,119 @@
|
||||||
|
From 95b15b3d2f180b15267032e16c947c0f9b8a112d Mon Sep 17 00:00:00 2001
|
||||||
|
From: Dave Airlie <airlied@redhat.com>
|
||||||
|
Date: Wed, 1 Mar 2017 13:02:38 +1000
|
||||||
|
Subject: [PATCH] Revert "Merging r280589:"
|
||||||
|
|
||||||
|
This reverts commit 25e2616626caafb896517e18cd8aa724fba2b200.
|
||||||
|
---
|
||||||
|
lib/Target/AMDGPU/SIInstructions.td | 1 -
|
||||||
|
lib/Target/AMDGPU/SIWholeQuadMode.cpp | 7 +++++
|
||||||
|
test/CodeGen/AMDGPU/wqm.ll | 49 +++--------------------------------
|
||||||
|
3 files changed, 11 insertions(+), 46 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/lib/Target/AMDGPU/SIInstructions.td b/lib/Target/AMDGPU/SIInstructions.td
|
||||||
|
index dde5f2f..18b7d5d 100644
|
||||||
|
--- a/lib/Target/AMDGPU/SIInstructions.td
|
||||||
|
+++ b/lib/Target/AMDGPU/SIInstructions.td
|
||||||
|
@@ -2029,7 +2029,6 @@ def SI_RETURN : PseudoInstSI <
|
||||||
|
let hasSideEffects = 1;
|
||||||
|
let SALU = 1;
|
||||||
|
let hasNoSchedulingInfo = 1;
|
||||||
|
- let DisableWQM = 1;
|
||||||
|
}
|
||||||
|
|
||||||
|
let Uses = [EXEC], Defs = [EXEC, VCC, M0],
|
||||||
|
diff --git a/lib/Target/AMDGPU/SIWholeQuadMode.cpp b/lib/Target/AMDGPU/SIWholeQuadMode.cpp
|
||||||
|
index 1534d58..b200c15 100644
|
||||||
|
--- a/lib/Target/AMDGPU/SIWholeQuadMode.cpp
|
||||||
|
+++ b/lib/Target/AMDGPU/SIWholeQuadMode.cpp
|
||||||
|
@@ -219,6 +219,13 @@ char SIWholeQuadMode::scanInstructions(MachineFunction &MF,
|
||||||
|
markInstruction(MI, Flags, Worklist);
|
||||||
|
GlobalFlags |= Flags;
|
||||||
|
}
|
||||||
|
+
|
||||||
|
+ if (WQMOutputs && MBB.succ_empty()) {
|
||||||
|
+ // This is a prolog shader. Make sure we go back to exact mode at the end.
|
||||||
|
+ Blocks[&MBB].OutNeeds = StateExact;
|
||||||
|
+ Worklist.push_back(&MBB);
|
||||||
|
+ GlobalFlags |= StateExact;
|
||||||
|
+ }
|
||||||
|
}
|
||||||
|
|
||||||
|
return GlobalFlags;
|
||||||
|
diff --git a/test/CodeGen/AMDGPU/wqm.ll b/test/CodeGen/AMDGPU/wqm.ll
|
||||||
|
index 41e4264..809a7ba 100644
|
||||||
|
--- a/test/CodeGen/AMDGPU/wqm.ll
|
||||||
|
+++ b/test/CodeGen/AMDGPU/wqm.ll
|
||||||
|
@@ -17,18 +17,17 @@ main_body:
|
||||||
|
;CHECK-LABEL: {{^}}test2:
|
||||||
|
;CHECK-NEXT: ; %main_body
|
||||||
|
;CHECK-NEXT: s_wqm_b64 exec, exec
|
||||||
|
+;CHECK: image_sample
|
||||||
|
;CHECK-NOT: exec
|
||||||
|
-define amdgpu_ps void @test2(<8 x i32> inreg %rsrc, <4 x i32> inreg %sampler, float addrspace(1)* inreg %ptr, <4 x i32> %c) {
|
||||||
|
+;CHECK: _load_dword v0,
|
||||||
|
+define amdgpu_ps float @test2(<8 x i32> inreg %rsrc, <4 x i32> inreg %sampler, float addrspace(1)* inreg %ptr, <4 x i32> %c) {
|
||||||
|
main_body:
|
||||||
|
%c.1 = call <4 x float> @llvm.SI.image.sample.v4i32(<4 x i32> %c, <8 x i32> %rsrc, <4 x i32> %sampler, i32 15, i32 0, i32 0, i32 0, i32 0, i32 0, i32 0, i32 0)
|
||||||
|
%c.2 = bitcast <4 x float> %c.1 to <4 x i32>
|
||||||
|
%c.3 = extractelement <4 x i32> %c.2, i32 0
|
||||||
|
%gep = getelementptr float, float addrspace(1)* %ptr, i32 %c.3
|
||||||
|
%data = load float, float addrspace(1)* %gep
|
||||||
|
-
|
||||||
|
- call void @llvm.SI.export(i32 15, i32 1, i32 1, i32 0, i32 1, float %data, float undef, float undef, float undef)
|
||||||
|
-
|
||||||
|
- ret void
|
||||||
|
+ ret float %data
|
||||||
|
}
|
||||||
|
|
||||||
|
; ... but disabled for stores (and, in this simple case, not re-enabled).
|
||||||
|
@@ -415,46 +414,6 @@ entry:
|
||||||
|
ret void
|
||||||
|
}
|
||||||
|
|
||||||
|
-; Must return to exact at the end of a non-void returning shader,
|
||||||
|
-; otherwise the EXEC mask exported by the epilog will be wrong. This is true
|
||||||
|
-; even if the shader has no kills, because a kill could have happened in a
|
||||||
|
-; previous shader fragment.
|
||||||
|
-;
|
||||||
|
-; CHECK-LABEL: {{^}}test_nonvoid_return:
|
||||||
|
-; CHECK: s_mov_b64 [[LIVE:s\[[0-9]+:[0-9]+\]]], exec
|
||||||
|
-; CHECK: s_wqm_b64 exec, exec
|
||||||
|
-;
|
||||||
|
-; CHECK: s_and_b64 exec, exec, [[LIVE]]
|
||||||
|
-; CHECK-NOT: exec
|
||||||
|
-define amdgpu_ps <4 x float> @test_nonvoid_return() nounwind {
|
||||||
|
- %tex = call <4 x float> @llvm.SI.image.sample.v4i32(<4 x i32> undef, <8 x i32> undef, <4 x i32> undef, i32 15, i32 0, i32 0, i32 0, i32 0, i32 0, i32 0, i32 0)
|
||||||
|
- %tex.i = bitcast <4 x float> %tex to <4 x i32>
|
||||||
|
- %dtex = call <4 x float> @llvm.SI.image.sample.v4i32(<4 x i32> %tex.i, <8 x i32> undef, <4 x i32> undef, i32 15, i32 0, i32 0, i32 0, i32 0, i32 0, i32 0, i32 0)
|
||||||
|
- ret <4 x float> %dtex
|
||||||
|
-}
|
||||||
|
-
|
||||||
|
-; CHECK-LABEL: {{^}}test_nonvoid_return_unreachable:
|
||||||
|
-; CHECK: s_mov_b64 [[LIVE:s\[[0-9]+:[0-9]+\]]], exec
|
||||||
|
-; CHECK: s_wqm_b64 exec, exec
|
||||||
|
-;
|
||||||
|
-; CHECK: s_and_b64 exec, exec, [[LIVE]]
|
||||||
|
-; CHECK-NOT: exec
|
||||||
|
-define amdgpu_ps <4 x float> @test_nonvoid_return_unreachable(i32 inreg %c) nounwind {
|
||||||
|
-entry:
|
||||||
|
- %tex = call <4 x float> @llvm.SI.image.sample.v4i32(<4 x i32> undef, <8 x i32> undef, <4 x i32> undef, i32 15, i32 0, i32 0, i32 0, i32 0, i32 0, i32 0, i32 0)
|
||||||
|
- %tex.i = bitcast <4 x float> %tex to <4 x i32>
|
||||||
|
- %dtex = call <4 x float> @llvm.SI.image.sample.v4i32(<4 x i32> %tex.i, <8 x i32> undef, <4 x i32> undef, i32 15, i32 0, i32 0, i32 0, i32 0, i32 0, i32 0, i32 0)
|
||||||
|
-
|
||||||
|
- %cc = icmp sgt i32 %c, 0
|
||||||
|
- br i1 %cc, label %if, label %else
|
||||||
|
-
|
||||||
|
-if:
|
||||||
|
- store volatile <4 x float> %dtex, <4 x float>* undef
|
||||||
|
- unreachable
|
||||||
|
-
|
||||||
|
-else:
|
||||||
|
- ret <4 x float> %dtex
|
||||||
|
-}
|
||||||
|
|
||||||
|
declare void @llvm.amdgcn.image.store.v4i32(<4 x float>, <4 x i32>, <8 x i32>, i32, i1, i1, i1, i1) #1
|
||||||
|
declare void @llvm.amdgcn.buffer.store.f32(float, <4 x i32>, i32, i32, i1, i1) #1
|
||||||
|
--
|
||||||
|
2.9.3
|
||||||
|
|
||||||
|
|
@ -7,7 +7,7 @@
|
||||||
|
|
||||||
Name: llvm
|
Name: llvm
|
||||||
Version: 3.9.1
|
Version: 3.9.1
|
||||||
Release: 2%{?dist}
|
Release: 3%{?dist}
|
||||||
Summary: The Low Level Virtual Machine
|
Summary: The Low Level Virtual Machine
|
||||||
|
|
||||||
License: NCSA
|
License: NCSA
|
||||||
|
|
@ -28,6 +28,8 @@ Patch5: 0001-cmake-Install-CheckAtomic.cmake-needed-by-lldb.patch
|
||||||
# Upstream patch to fix doc build
|
# Upstream patch to fix doc build
|
||||||
# http://llvm.org/viewvc/llvm-project?view=revision&revision=294646
|
# http://llvm.org/viewvc/llvm-project?view=revision&revision=294646
|
||||||
Patch6: llvm-r294646.patch
|
Patch6: llvm-r294646.patch
|
||||||
|
# This fix caused regressions
|
||||||
|
Patch7: 0001-Revert-Merging-r280589.patch
|
||||||
|
|
||||||
# backports cribbed from https://github.com/rust-lang/llvm/
|
# backports cribbed from https://github.com/rust-lang/llvm/
|
||||||
Patch47: rust-lang-llvm-pr47.patch
|
Patch47: rust-lang-llvm-pr47.patch
|
||||||
|
|
@ -93,6 +95,7 @@ Static libraries for the LLVM compiler infrastructure.
|
||||||
%patch4 -p1 -b .docs4
|
%patch4 -p1 -b .docs4
|
||||||
%patch5 -p1 -b .lldbfix
|
%patch5 -p1 -b .lldbfix
|
||||||
%patch6 -p0 -b .doc-lit
|
%patch6 -p0 -b .doc-lit
|
||||||
|
%patch7 -p1 -b .amdfix
|
||||||
%patch47 -p1 -b .rust47
|
%patch47 -p1 -b .rust47
|
||||||
%patch53 -p1 -b .rust53
|
%patch53 -p1 -b .rust53
|
||||||
%patch54 -p1 -b .rust54
|
%patch54 -p1 -b .rust54
|
||||||
|
|
@ -214,6 +217,9 @@ make check-all || :
|
||||||
%{_libdir}/*.a
|
%{_libdir}/*.a
|
||||||
|
|
||||||
%changelog
|
%changelog
|
||||||
|
* Wed Mar 01 2017 Dave Airlie <airlied@redhat.com> - 3.9.1-3
|
||||||
|
- revert upstream radeonsi breaking change.
|
||||||
|
|
||||||
* Thu Feb 23 2017 Josh Stone <jistone@redhat.com> - 3.9.1-2
|
* Thu Feb 23 2017 Josh Stone <jistone@redhat.com> - 3.9.1-2
|
||||||
- disable sphinx warnings-as-errors
|
- disable sphinx warnings-as-errors
|
||||||
|
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue