[PATCH] D130964: [X86][BF16] Enable __bf16 for x86 targets.

2022-08-09 Thread Phoebe Wang via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGe4888a37d367: [X86][BF16] Enable __bf16 for x86 targets. (authored by FreddyYe, committed by pengfei). Changed prior to commit:

[PATCH] D130964: [X86][BF16] Enable __bf16 for x86 targets.

2022-08-04 Thread Phoebe Wang via Phabricator via cfe-commits
pengfei accepted this revision. pengfei added a comment. This revision is now accepted and ready to land. LGTM. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D130964/new/ https://reviews.llvm.org/D130964

[PATCH] D130964: [X86][BF16] Enable __bf16 for x86 targets.

2022-08-03 Thread Freddy, Ye via Phabricator via cfe-commits
FreddyYe added a comment. > Right, but this patch is adding x86 support whenever SSE2 is available. > AVX512BF16 is available on a *very* small slice of processors. In contrast, > e.g. F16C is relatively broadly available, although I understand that we > formally support `_Float16` all the

[PATCH] D130964: [X86][BF16] Enable __bf16 for x86 targets.

2022-08-03 Thread John McCall via Phabricator via cfe-commits
rjmccall added a comment. In D130964#3695408 , @FreddyYe wrote: > In D130964#3694540 , @bkramer wrote: > >> In D130964#3694473 , @rjmccall >> wrote: >> >>> How are you

[PATCH] D130964: [X86][BF16] Enable __bf16 for x86 targets.

2022-08-02 Thread Freddy, Ye via Phabricator via cfe-commits
FreddyYe added a comment. In D130964#3694540 , @bkramer wrote: > In D130964#3694473 , @rjmccall > wrote: > >> How are you actually implementing `__bf16` on these targets? There isn't >> even hardware support

[PATCH] D130964: [X86][BF16] Enable __bf16 for x86 targets.

2022-08-02 Thread Phoebe Wang via Phabricator via cfe-commits
pengfei added a comment. In 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`.

[PATCH] D130964: [X86][BF16] Enable __bf16 for x86 targets.

2022-08-02 Thread Benjamin Kramer via Phabricator via cfe-commits
bkramer added a comment. In D130964#3694473 , @rjmccall wrote: > How are you actually implementing `__bf16` on these targets? There isn't > even hardware support for conversions. `bf16` -> `float` is really just a bit shift. The other direction gets

[PATCH] D130964: [X86][BF16] Enable __bf16 for x86 targets.

2022-08-02 Thread John McCall via Phabricator via cfe-commits
rjmccall added a comment. How are you actually implementing `__bf16` on these targets? There isn't even hardware support for conversions. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D130964/new/ https://reviews.llvm.org/D130964

[PATCH] D130964: [X86][BF16] Enable __bf16 for x86 targets.

2022-08-02 Thread Phoebe Wang via Phabricator via cfe-commits
pengfei added a comment. Add to `ReleaseNotes.rst` as well. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D130964/new/ https://reviews.llvm.org/D130964 ___ cfe-commits mailing list