rjodinchr wrote:

> CC @rjodinchr. I realise now that this idea may prove problematic for 
> `clspv`/`clspv64` targets. If this idea were to taken further, things like 
> OpenCL `minmag` would call `__clc_minmag` which would call `__clc_fabs` 
> (e.g.), but I notice `fabs` is not implemented in `clspv*` libclc and it's 
> left as external declarations. What exactly is happening with clspv libclc 
> builtins? Same question for conversions, etc.

`clspv` has a native support for `fabs`, it does not need libclc to define that 
function. But as I understand that change, maybe `clspv` will need to add alias 
for some `__clc_<something>`, but it should not be a big issue I believe.

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

Reply via email to