[PATCH] D37985: [AMDGPU] add LDS f32 intrinsics

[Matt Arsenault via Phabricator via llvm-commits]

arsenm accepted this revision.
arsenm added a comment.

LGTM except for the mystery name mangling in the test



================
Comment at: test/CodeGen/AMDGPU/lds_atomic_f32.ll:4
+
+declare float @llvm.amdgcn.atomic.fadd.f32(float addrspace(3)* nocapture, float, i32, i32, i1)
+declare float @llvm.amdgcn.atomic.fmin.f32(float addrspace(3)* nocapture, float, i32, i32, i1)
----------------
These weren't declared as mangled, so why do these have the .f32? I'm kind of surprised this is accepted


https://reviews.llvm.org/D37985