[PATCH] D105331: [CFE][X86] Enable complex _Float16.

Pengfei Wang via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 2 02:04:03 PDT 2021


pengfei added a comment.

In D105331#2854934 <https://reviews.llvm.org/D105331#2854934>, @SjoerdMeijer wrote:

> The patch on itself looks reasonable to me. I was just wondering about the _Float16 support on X86 in general because the Clang docs says:
>
>   _Float16 is currently only supported on the following targets, with further targets pending ABI standardization: <list of non-X86 targets>
>
> Does X86 support _Float16, and how does that work for Complex _Float16?  
> Either way, does this need a doc update too?

Good point! Yes, X86 is going to support _Float16. This patch is split from D105263 <https://reviews.llvm.org/D105263> and depended on it. I will update the doc in that patch. Thanks.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105331



More information about the cfe-commits mailing list