[AMDGPU] Pre-commit global-isel test case for D106451
authorJay Foad <jay.foad@amd.com>
Wed, 21 Jul 2021 10:29:04 +0000 (11:29 +0100)
committerJay Foad <jay.foad@amd.com>
Mon, 26 Jul 2021 13:27:30 +0000 (14:27 +0100)
This test case shows the scheduler wrongly reordering two buffer
accesses that might alias.

llvm/test/CodeGen/AMDGPU/GlobalISel/buffer-schedule.ll [new file with mode: 0644]

diff --git a/llvm/test/CodeGen/AMDGPU/GlobalISel/buffer-schedule.ll b/llvm/test/CodeGen/AMDGPU/GlobalISel/buffer-schedule.ll
new file mode 100644 (file)
index 0000000..cbfb47d
--- /dev/null
@@ -0,0 +1,22 @@
+; RUN: llc -global-isel -mtriple=amdgcn-amd-amdpal -mcpu=gfx900 -verify-machineinstrs < %s | FileCheck -check-prefixes=GCN %s
+
+; FIXME: the first load and store should not be reordered because they might
+; alias depending on the value of %off
+; GCN-LABEL: {{^}}test1:
+; GCN: buffer_load_dword
+; GCN: buffer_store_dword
+; GCN: buffer_store_dword
+define amdgpu_cs void @test1(<4 x i32> inreg %buf, i32 %off) {
+.entry:
+  call void @llvm.amdgcn.raw.buffer.store.i32(i32 0, <4 x i32> %buf, i32 8, i32 0, i32 0)
+  %val = call i32 @llvm.amdgcn.raw.buffer.load.i32(<4 x i32> %buf, i32 %off, i32 0, i32 0)
+  call void @llvm.amdgcn.raw.buffer.store.i32(i32 %val, <4 x i32> %buf, i32 0, i32 0, i32 0)
+  ret void
+}
+
+declare i32 @llvm.amdgcn.raw.buffer.load.i32(<4 x i32>, i32, i32, i32) #2
+
+declare void @llvm.amdgcn.raw.buffer.store.i32(i32, <4 x i32>, i32, i32, i32) #3
+
+attributes #2 = { nounwind readonly }
+attributes #3 = { nounwind writeonly }