R600 intrinsics patch
Matt Arsenault
arsenm2 at gmail.com
Tue Mar 17 11:03:11 PDT 2015
> On Mar 17, 2015, at 10:09 AM, Tom Stellard <tom at stellard.net> wrote:
>
> On Tue, Mar 17, 2015 at 09:53:08AM -0700, Matt Arsenault wrote:
>> On 03/17/2015 08:11 AM, Tom Stellard wrote:
>>> The target prefix for all these has been changed from AMDGPU to amdgpu.
>>> I don't think this is necessary. Matt may know more about this.
>> I don't remember all the details, but this is basically necessary for
>> the automatic support from GCCBuiltin to work properly. The current set
>> of builtins handled in clang need custom handling for things like
>> conversion to/from i1, converting struct returns to out arguments, and
>> handling overloads so they require custom code anyway. For simpler
>> builtins, to not need custom code needs the target prefix to match.
>
>
> What does the target prefix need to match? The triple?
>
> -Tom
Almost. It needs to match what getArchTypePrefix on the triple gives
More information about the llvm-commits
mailing list