farzonl wrote:

> I'm worried if we add a whole new series of math intrinsics without adding 
> the corresponding constrained intrinsics, it's going to be confusing for 
> anyone trying to understand how math intrinsics work. Why does tan exist, but 
> not constrained tan?
> 
> `tan` is defined basically the same way as `sin` in Builtins.td; the only 
> difference is that instead of `def SinF16F128 : Builtin, 
> F16F128MathTemplate`, there's just `def TanF128 : Builtin {` (so no f16 
> support).

@efriedma-quic Would you be open to me making a new issue to support 
constrained intrinsics. I'll need to spend some time to see what's required to 
support them and I didn't want the current set of changes to go stale. 

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

Reply via email to