AMDGPU/GlobalISel: Move llvm.amdgcn.raw.buffer.store handling
ClosedPublic
Actions

Authored by arsenm on Jan 13 2020, 7:46 PM.

Details

Reviewers

nhaehnle
kerbowa

Summary

Treat this the same way as loads. There's less value to the
intermediate nodes, but it's good to be consistent.

LGTM

llvm/lib/Target/AMDGPU/SIInstructions.td
2119	extra newline

This revision is now accepted and ready to land.Jan 18 2020, 8:03 PM

			Path	Packages
M			llvm/lib/Target/AMDGPU/AMDGPUGISel.td (5 lines)
M			llvm/lib/Target/AMDGPU/AMDGPUInstructionSelector.h (4 lines)
M			llvm/lib/Target/AMDGPU/AMDGPUInstructionSelector.cpp (229 lines)
M			llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.h (12 lines)
M			llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp (118 lines)
M			llvm/lib/Target/AMDGPU/AMDGPURegisterBankInfo.cpp (14 lines)
M			llvm/lib/Target/AMDGPU/BUFInstructions.td (18 lines)
M			llvm/lib/Target/AMDGPU/SIInstructions.td (27 lines)
M			llvm/test/CodeGen/AMDGPU/GlobalISel/llvm.amdgcn.raw.buffer.store.format.f16.ll (78 lines)
M			llvm/test/CodeGen/AMDGPU/GlobalISel/llvm.amdgcn.raw.buffer.store.format.f32.ll (32 lines)
M			llvm/test/CodeGen/AMDGPU/GlobalISel/llvm.amdgcn.raw.buffer.store.ll (65 lines)

Status	Author	Revision
Accepted	arsenm	D58233 Allow replacing intrinsic operands with variables
Closed	arsenm	D58351 ARM: Add ImmArg to intrinsics
Closed	arsenm	D58352 Mips: Add ImmArg to intrinsics
Closed	arsenm	D58353 SystemZ: Add ImmArg to intrinsics
Closed	arsenm	D58354 PowerPC: Add ImmArg to intrinsics
Closed	arsenm	D58356 Hexagon: Add ImmArg to intrinsics
Closed	craig.topper	D58302 [X86] Add ImmArg markings to intrinsics.
Needs Review	arsenm	D58449 Junk: Add assert to find GCCBuiltins with constant arguments missing
Closed	arsenm	D64877 AMDGPU/GlobalISel: Attempt to RegBankSelect image intrinsics
Closed	arsenm	D67045 AMDGPU/GlobalISel: Select llvm.amdgcn.raw.buffer.store.format
Closed	arsenm	D58232 GlobalISel: Don't materialize immarg arguments to intrinsics
Closed	arsenm	D57825 IR: Add immarg attribute
Closed	arsenm	D64755 ARM: Fix missing immarg for space intrinsic
Closed	arsenm	D64778 Mips: Remove immarg from copy and insert intrinsics
Closed	arsenm	D72677 AMDGPU/GlobalISel: Move llvm.amdgcn.raw.buffer.store handling
Closed	arsenm	D65238 AMDGPU/GlobalISel: Select llvm.amdgcn.raw.buffer.store