[llvm] [PATCH 2/2] AMDGPU: Add image attribute reader intrinsic.

[Matt Arsenault]

> On Jul 10, 2015, at 2:08 PM, Tom Stellard <tom at stellard.net> wrote:
> 
> This should use the amdgpu target prefix.

It actually shouldn’t. The current situation is kind of broken. We have a mix of AMDIL, AMDGPU, r600 and SI. These are supposed to match the target arch name as in the triple. We really should have __builtin_r600 _and __builtin_amdgcn_ but right now most are using __builtin_amdgpu/llvm.AMDGPU

-Matt
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150710/395e6a83/attachment.html>