pengfei added a comment.

In D130964#3694473 <https://reviews.llvm.org/D130964#3694473>, @rjmccall wrote:

> How are you actually implementing `__bf16` on these targets?  There isn't 
> even hardware support for conversions.

We support `float` -> `bf16` in `AVX512BF16`. 
https://www.intel.com/content/www/us/en/docs/intrinsics-guide/index.html#avx512techs=AVX512_BF16
And we found some problems in how to represent `bf16` types in intrinsics. For 
example, we currently defined `__bfloat16` as `unsigned short`. We cannot stop 
user e.g., adding 2 `__bfloat16` in C code and getting the wrong result. So we 
want to introduce the type on X86. For more information, please see the 
discussions in D120395 <https://reviews.llvm.org/D120395>,


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D130964/new/

https://reviews.llvm.org/D130964

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to