[PATCH] D119926: [Clang][AArch64] Enable _Float16 _Complex type

MattDevereau abandoned this revision. MattDevereau added a comment. Abandoning this patch as it is redundant due to changes in D105331 . This patch also drew attention to unused code removed in 841355c1e4e35fc02b5b171419979f5f9af0ebc8

[PATCH] D119926: [Clang][AArch64] Enable _Float16 _Complex type

aaron.ballman added inline comments. Comment at: clang/test/CodeGen/aarch64-complex-half-math.c:3 +// RUN: %clang_cc1 %s -emit-llvm -triple aarch64-unknown-unknown -o - | FileCheck %s --check-prefix=AARCH64 +// REQUIRES: aarch64-registered-target +

[PATCH] D119926: [Clang][AArch64] Enable _Float16 _Complex type

peterwaller-arm added inline comments. Comment at: clang/test/CodeGen/aarch64-complex-half-math.c:3 +// RUN: %clang_cc1 %s -emit-llvm -triple aarch64-unknown-unknown -o - | FileCheck %s --check-prefix=AARCH64 +// REQUIRES: aarch64-registered-target +

[PATCH] D119926: [Clang][AArch64] Enable _Float16 _Complex type

aaron.ballman added inline comments. Comment at: clang/test/CodeGen/aarch64-complex-half-math.c:3 +// RUN: %clang_cc1 %s -emit-llvm -triple aarch64-unknown-unknown -o - | FileCheck %s --check-prefix=AARCH64 +// REQUIRES: aarch64-registered-target + There's

[PATCH] D119926: [Clang][AArch64] Enable _Float16 _Complex type

peterwaller-arm accepted this revision. peterwaller-arm added a comment. This revision is now accepted and ready to land. LGTM. Comment at: clang/test/CodeGen/aarch64-complex-half-math.c:5 + // AARCH64-LABEL: @add_float_rr( + // AARCH64: fadd reassoc nnan ninf nsz arcp afn

[PATCH] D119926: [Clang][AArch64] Enable _Float16 _Complex type

MattDevereau updated this revision to Diff 409236. MattDevereau added a comment. Removed -O1 and -ffast-math flags Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D119926/new/ https://reviews.llvm.org/D119926 Files: clang/lib/AST/ASTContext.cpp

[PATCH] D119926: [Clang][AArch64] Enable _Float16 _Complex type

aaron.ballman added inline comments. Comment at: clang/include/clang/AST/ASTContext.h:1117 CanQualType BFloat16Ty; - CanQualType Float16Ty; // C11 extension ISO/IEC TS 18661-3 + CanQualType Float16Ty, Float16ComplexTy; // C11 extension ISO/IEC TS 18661-3 CanQualType

[PATCH] D119926: [Clang][AArch64] Enable _Float16 _Complex type

fhahn added a reviewer: aaron.ballman. fhahn added inline comments. Comment at: clang/test/CodeGen/aarch64-complex-half-math.c:1 +// RUN: %clang_cc1 %s -O1 -emit-llvm -triple aarch64-unknown-unknown -ffast-math -o - | FileCheck %s --check-prefix=AARCH64 +

[PATCH] D119926: [Clang][AArch64] Enable _Float16 _Complex type

peterwaller-arm added a comment. Some comments. Comment at: clang/include/clang/AST/ASTContext.h:1117 CanQualType BFloat16Ty; - CanQualType Float16Ty; // C11 extension ISO/IEC TS 18661-3 + CanQualType Float16Ty, Float16ComplexTy; // C11 extension ISO/IEC TS 18661-3

[PATCH] D119926: [Clang][AArch64] Enable _Float16 _Complex type

paulwalker-arm added inline comments. Comment at: clang/include/clang/AST/ASTContext.h:1117 CanQualType BFloat16Ty; - CanQualType Float16Ty; // C11 extension ISO/IEC TS 18661-3 + CanQualType Float16Ty, Float16ComplexTy; // C11 extension ISO/IEC TS 18661-3 CanQualType

[PATCH] D119926: [Clang][AArch64] Enable _Float16 _Complex type

MattDevereau created this revision. MattDevereau added reviewers: georges, peterwaller-arm, paulwalker-arm, DavidTruby, bsmith, teemperor, rjmccall, qiucf. Herald added a subscriber: kristof.beyls. MattDevereau requested review of this revision. Herald added a project: clang. Herald added a