arsenm wrote:

> There are targets that use a different integer to denote flat (e.g. see SPIR 
> & SPIR-V). Whilst I know that there are objections to that, the fact remains 
> that they had historical reason (wanted to make legacy OCL convention that 
> the default is private work, and given that IR defaults to 0 this was an 
> easy, if possibly costly, way out; 

The SPIRV IR would be better off changing its numbers around like we did in 
AMDGPU ages ago. The only concern would be bitcode compatibility, but given 
it's still an "experimental target" that shouldn't be an issue.

> AMDGPU also borks this for legacy OCL reasons, which has been a source of 
> pain). 

This is only a broken in-clang hack, the backend IR always uses the correct 
address space 

https://github.com/llvm/llvm-project/pull/108786
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to