mirror of
https://github.com/RPCS3/llvm-mirror.git
synced 2024-10-24 05:23:45 +02:00
048a920e0e
Summary: The goal is for each operand type to have its own parse function and at the same time share common code for tracking state as different instruction types share operand types (e.g. glc/glc_flat, etc). Introduce parseAMDGPUOperand which can parse any optional operand. DPP and Clamp/OMod have custom handling for now. Sam also suggested to have class hierarchy for operand types instead of table. This can be done in separate change. Remove parseVOP3OptionalOps, parseDS*OptionalOps, parseFlatOptionalOps, parseMubufOptionalOps, parseDPPOptionalOps. Reduce number of definitions of AsmOperand's and MatchClasses' by using common base class. Rename AsmMatcher/InstPrinter methods accordingly. Print immediate type when printing parsed immediate operand. Use 'off' if offset/index register is unused instead of skipping it to make it more readable (also agreed with SP3). Update tests. Reviewers: tstellarAMD, SamWot, artem.tamazov Subscribers: qcolombet, arsenm, llvm-commits Differential Revision: http://reviews.llvm.org/D19584 llvm-svn: 268015
40 lines
1.6 KiB
LLVM
40 lines
1.6 KiB
LLVM
; RUN: llc -march=amdgcn -verify-machineinstrs < %s | FileCheck -check-prefix=FUNC -check-prefix=SI %s
|
|
|
|
|
|
; FIXME: This currently doesn't do a great job of clustering the
|
|
; loads, which end up with extra moves between them. Right now, it
|
|
; seems the only things areLoadsFromSameBasePtr is accomplishing is
|
|
; ordering the loads so that the lower address loads come first.
|
|
|
|
; FUNC-LABEL: {{^}}cluster_global_arg_loads:
|
|
; SI-DAG: buffer_load_dword [[REG0:v[0-9]+]], off, s{{\[[0-9]+:[0-9]+\]}}, 0{{$}}
|
|
; SI-DAG: buffer_load_dword [[REG1:v[0-9]+]], off, s{{\[[0-9]+:[0-9]+\]}}, 0 offset:4
|
|
; SI: buffer_store_dword [[REG0]]
|
|
; SI: buffer_store_dword [[REG1]]
|
|
define void @cluster_global_arg_loads(i32 addrspace(1)* %out0, i32 addrspace(1)* %out1, i32 addrspace(1)* %ptr) #0 {
|
|
%load0 = load i32, i32 addrspace(1)* %ptr, align 4
|
|
%gep = getelementptr i32, i32 addrspace(1)* %ptr, i32 1
|
|
%load1 = load i32, i32 addrspace(1)* %gep, align 4
|
|
store i32 %load0, i32 addrspace(1)* %out0, align 4
|
|
store i32 %load1, i32 addrspace(1)* %out1, align 4
|
|
ret void
|
|
}
|
|
|
|
; Test for a crach in SIInstrInfo::areLoadsFromSameBasePtr() when checking
|
|
; an MUBUF load which does not have a vaddr operand.
|
|
; FUNC-LABEL: {{^}}same_base_ptr_crash:
|
|
; SI: buffer_load_dword
|
|
; SI: buffer_load_dword
|
|
define void @same_base_ptr_crash(i32 addrspace(1)* %out, i32 addrspace(1)* %in, i32 %offset) {
|
|
entry:
|
|
%out1 = getelementptr i32, i32 addrspace(1)* %out, i32 %offset
|
|
%tmp0 = load i32, i32 addrspace(1)* %out
|
|
%tmp1 = load i32, i32 addrspace(1)* %out1
|
|
%tmp2 = add i32 %tmp0, %tmp1
|
|
store i32 %tmp2, i32 addrspace(1)* %out
|
|
ret void
|
|
}
|
|
|
|
attributes #0 = { nounwind }
|
|
attributes #1 = { nounwind readnone }
|