karolherbst wrote:

> I have added back __clc_runtime_has_hw_fma32 (deleted in 
> [39f3d72](https://github.com/llvm/llvm-project/commit/39f3d72b72fdb6831886148f1172d2f5b0ab2d49)).
>  It acts as a global control. By default it is disabled. You can replace its 
> return value to true for targets that has hw fma. So it should be easier for 
> you to control both _Z3fmafff and _Z10__clc_erfcf.

Yeah, I think that's generally a good idea, because I've seen some builtins 
that use `fma` unconditionally and I think being able to provide an alternative 
implementation for hw not having `fma` is a useful thing to have. And if that 
helper can be used within libclc then that's good.

I still have to wire up overriding those helper functions (including the denorm 
ones), but that's something I wanted to look into pretty soon anyway.

https://github.com/llvm/llvm-project/pull/199626
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to