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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
