[PATCH] D73185: [AST] Add fixed-point subtraction constant evaluation.
This revision was automatically updated to reflect the committed changes. Closed by commit rGeccf7fc7b31a: [AST] Add fixed-point subtraction constant evaluation. (authored by ebevhan). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73185/new/ https://reviews.llvm.org/D73185 Files: clang/include/clang/Basic/FixedPoint.h clang/lib/AST/ExprConstant.cpp clang/lib/Basic/FixedPoint.cpp clang/test/Frontend/fixed_point_sub.c Index: clang/test/Frontend/fixed_point_sub.c === --- clang/test/Frontend/fixed_point_sub.c +++ clang/test/Frontend/fixed_point_sub.c @@ -1,6 +1,55 @@ // RUN: %clang_cc1 -ffixed-point -S -emit-llvm %s -o - | FileCheck %s --check-prefixes=CHECK,SIGNED // RUN: %clang_cc1 -ffixed-point -fpadding-on-unsigned-fixed-point -S -emit-llvm %s -o - | FileCheck %s --check-prefixes=CHECK,UNSIGNED +// Subtraction between different fixed point types +short _Accum sa_const = 1.0hk - 2.0hk; // CHECK-DAG: @sa_const = {{.*}}global i16 -128, align 2 +_Accum a_const = 1.0hk - 2.0k; // CHECK-DAG: @a_const = {{.*}}global i32 -32768, align 4 +long _Accum la_const = 1.0hk - 2.0lk; // CHECK-DAG: @la_const = {{.*}}global i64 -2147483648, align 8 +short _Accum sa_const2 = 0.5hr - 2.0hk; // CHECK-DAG: @sa_const2 = {{.*}}global i16 -192, align 2 +short _Accum sa_const3 = 0.5r - 2.0hk; // CHECK-DAG: @sa_const3 = {{.*}}global i16 -192, align 2 +short _Accum sa_const4 = 0.5lr - 2.0hk; // CHECK-DAG: @sa_const4 = {{.*}}global i16 -192, align 2 +short _Accum sa_const5 = 2.0hk - 0.5lr; // CHECK-DAG: @sa_const5 = {{.*}}global i16 192, align 2 + +// Unsigned subtraction +unsigned short _Accum usa_const = 3.0uhk - 2.0uhk; +// CHECK-SIGNED-DAG: @usa_const = {{.*}}global i16 768, align 2 +// CHECK-UNSIGNED-DAG: @usa_const = {{.*}}global i16 384, align 2 + +// Unsigned - signed +short _Accum sa_const6 = 1.0uhk - 2.0hk; +// CHECK-DAG: @sa_const6 = {{.*}}global i16 -128, align 2 + +// Subtraction with negative number +short _Accum sa_const7 = 0.5hr - (-2.0hk); +// CHECK-DAG: @sa_const7 = {{.*}}global i16 320, align 2 + +// Int subtraction +unsigned short _Accum usa_const2 = 2 - 0.5uhk; +// CHECK-SIGNED-DAG: @usa_const2 = {{.*}}global i16 640, align 2 +// CHECK-UNSIGNED-DAG: @usa_const2 = {{.*}}global i16 320, align 2 +short _Accum sa_const8 = 2 - (-0.5hk); // CHECK-DAG: @sa_const8 = {{.*}}global i16 320, align 2 +short _Accum sa_const9 = 257 - 2.0hk;// CHECK-DAG: @sa_const9 = {{.*}}global i16 32640, align 2 +long _Fract lf_const = 0.5lr - 1;// CHECK-DAG: @lf_const = {{.*}}global i32 -1073741824, align 4 + +// Saturated subtraction +_Sat short _Accum sat_sa_const = (_Sat short _Accum)128.0hk - (-128.0hk); +// CHECK-DAG: @sat_sa_const = {{.*}}global i16 32767, align 2 +_Sat unsigned short _Accum sat_usa_const = (_Sat unsigned short _Accum)128.0uhk - (-128.0uhk); +// CHECK-SIGNED-DAG: @sat_usa_const = {{.*}}global i16 65535, align 2 +// CHECK-UNSIGNED-DAG: @sat_usa_const = {{.*}}global i16 32767, align 2 +_Sat short _Accum sat_sa_const2 = (_Sat short _Accum)128.0hk - (-128); +// CHECK-DAG: @sat_sa_const2 = {{.*}}global i16 32767, align 2 +_Sat unsigned short _Accum sat_usa_const2 = (_Sat unsigned short _Accum)128.0uhk - (-128); +// CHECK-SIGNED-DAG: @sat_usa_const2 = {{.*}}global i16 65535, align 2 +// CHECK-UNSIGNED-DAG: @sat_usa_const2 = {{.*}}global i16 32767, align 2 +_Sat unsigned short _Accum sat_usa_const3 = (_Sat unsigned short _Accum)0.5uhk - 2; +// CHECK-DAG: @sat_usa_const3 = {{.*}}global i16 0, align 2 +_Sat short _Accum sat_sa_const3 = (_Sat short _Accum)-128.0hk - 128; +// CHECK-DAG: @sat_sa_const3 = {{.*}}global i16 -32768, align 2 +_Sat short _Accum sat_sa_const4 = (_Sat short _Accum)-150.0hk - 130.0lk; +// CHECK-DAG: @sat_sa_const4 = {{.*}}global i16 -32768, align 2 + + void SignedSubtraction() { // CHECK-LABEL: SignedSubtraction short _Accum sa; Index: clang/lib/Basic/FixedPoint.cpp === --- clang/lib/Basic/FixedPoint.cpp +++ clang/lib/Basic/FixedPoint.cpp @@ -173,6 +173,30 @@ return APFixedPoint(Result, CommonFXSema); } +APFixedPoint APFixedPoint::sub(const APFixedPoint &Other, + bool *Overflow) const { + auto CommonFXSema = Sema.getCommonSemantics(Other.getSemantics()); + APFixedPoint ConvertedThis = convert(CommonFXSema); + APFixedPoint ConvertedOther = Other.convert(CommonFXSema); + llvm::APSInt ThisVal = ConvertedThis.getValue(); + llvm::APSInt OtherVal = ConvertedOther.getValue(); + bool Overflowed = false; + + llvm::APSInt Result; + if (CommonFXSema.isSaturated()) { +Result = CommonFXSema.isSigned() ? ThisVal.ssub_sat(OtherVal) + : ThisVal.usub_sat(OtherVal); + } else { +Result = ThisVal.isSigned() ? ThisVal.ssub_ov(OtherVal, Overflowed) +: ThisVal.usub_ov(Oth
[PATCH] D73185: [AST] Add fixed-point subtraction constant evaluation.
ebevhan updated this revision to Diff 255975. ebevhan added a comment. Rebased. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73185/new/ https://reviews.llvm.org/D73185 Files: clang/include/clang/Basic/FixedPoint.h clang/lib/AST/ExprConstant.cpp clang/lib/Basic/FixedPoint.cpp clang/test/Frontend/fixed_point_sub.c Index: clang/test/Frontend/fixed_point_sub.c === --- clang/test/Frontend/fixed_point_sub.c +++ clang/test/Frontend/fixed_point_sub.c @@ -1,6 +1,55 @@ // RUN: %clang_cc1 -ffixed-point -S -emit-llvm %s -o - | FileCheck %s --check-prefixes=CHECK,SIGNED // RUN: %clang_cc1 -ffixed-point -fpadding-on-unsigned-fixed-point -S -emit-llvm %s -o - | FileCheck %s --check-prefixes=CHECK,UNSIGNED +// Subtraction between different fixed point types +short _Accum sa_const = 1.0hk - 2.0hk; // CHECK-DAG: @sa_const = {{.*}}global i16 -128, align 2 +_Accum a_const = 1.0hk - 2.0k; // CHECK-DAG: @a_const = {{.*}}global i32 -32768, align 4 +long _Accum la_const = 1.0hk - 2.0lk; // CHECK-DAG: @la_const = {{.*}}global i64 -2147483648, align 8 +short _Accum sa_const2 = 0.5hr - 2.0hk; // CHECK-DAG: @sa_const2 = {{.*}}global i16 -192, align 2 +short _Accum sa_const3 = 0.5r - 2.0hk; // CHECK-DAG: @sa_const3 = {{.*}}global i16 -192, align 2 +short _Accum sa_const4 = 0.5lr - 2.0hk; // CHECK-DAG: @sa_const4 = {{.*}}global i16 -192, align 2 +short _Accum sa_const5 = 2.0hk - 0.5lr; // CHECK-DAG: @sa_const5 = {{.*}}global i16 192, align 2 + +// Unsigned subtraction +unsigned short _Accum usa_const = 3.0uhk - 2.0uhk; +// CHECK-SIGNED-DAG: @usa_const = {{.*}}global i16 768, align 2 +// CHECK-UNSIGNED-DAG: @usa_const = {{.*}}global i16 384, align 2 + +// Unsigned - signed +short _Accum sa_const6 = 1.0uhk - 2.0hk; +// CHECK-DAG: @sa_const6 = {{.*}}global i16 -128, align 2 + +// Subtraction with negative number +short _Accum sa_const7 = 0.5hr - (-2.0hk); +// CHECK-DAG: @sa_const7 = {{.*}}global i16 320, align 2 + +// Int subtraction +unsigned short _Accum usa_const2 = 2 - 0.5uhk; +// CHECK-SIGNED-DAG: @usa_const2 = {{.*}}global i16 640, align 2 +// CHECK-UNSIGNED-DAG: @usa_const2 = {{.*}}global i16 320, align 2 +short _Accum sa_const8 = 2 - (-0.5hk); // CHECK-DAG: @sa_const8 = {{.*}}global i16 320, align 2 +short _Accum sa_const9 = 257 - 2.0hk;// CHECK-DAG: @sa_const9 = {{.*}}global i16 32640, align 2 +long _Fract lf_const = 0.5lr - 1;// CHECK-DAG: @lf_const = {{.*}}global i32 -1073741824, align 4 + +// Saturated subtraction +_Sat short _Accum sat_sa_const = (_Sat short _Accum)128.0hk - (-128.0hk); +// CHECK-DAG: @sat_sa_const = {{.*}}global i16 32767, align 2 +_Sat unsigned short _Accum sat_usa_const = (_Sat unsigned short _Accum)128.0uhk - (-128.0uhk); +// CHECK-SIGNED-DAG: @sat_usa_const = {{.*}}global i16 65535, align 2 +// CHECK-UNSIGNED-DAG: @sat_usa_const = {{.*}}global i16 32767, align 2 +_Sat short _Accum sat_sa_const2 = (_Sat short _Accum)128.0hk - (-128); +// CHECK-DAG: @sat_sa_const2 = {{.*}}global i16 32767, align 2 +_Sat unsigned short _Accum sat_usa_const2 = (_Sat unsigned short _Accum)128.0uhk - (-128); +// CHECK-SIGNED-DAG: @sat_usa_const2 = {{.*}}global i16 65535, align 2 +// CHECK-UNSIGNED-DAG: @sat_usa_const2 = {{.*}}global i16 32767, align 2 +_Sat unsigned short _Accum sat_usa_const3 = (_Sat unsigned short _Accum)0.5uhk - 2; +// CHECK-DAG: @sat_usa_const3 = {{.*}}global i16 0, align 2 +_Sat short _Accum sat_sa_const3 = (_Sat short _Accum)-128.0hk - 128; +// CHECK-DAG: @sat_sa_const3 = {{.*}}global i16 -32768, align 2 +_Sat short _Accum sat_sa_const4 = (_Sat short _Accum)-150.0hk - 130.0lk; +// CHECK-DAG: @sat_sa_const4 = {{.*}}global i16 -32768, align 2 + + void SignedSubtraction() { // CHECK-LABEL: SignedSubtraction short _Accum sa; Index: clang/lib/Basic/FixedPoint.cpp === --- clang/lib/Basic/FixedPoint.cpp +++ clang/lib/Basic/FixedPoint.cpp @@ -173,6 +173,30 @@ return APFixedPoint(Result, CommonFXSema); } +APFixedPoint APFixedPoint::sub(const APFixedPoint &Other, + bool *Overflow) const { + auto CommonFXSema = Sema.getCommonSemantics(Other.getSemantics()); + APFixedPoint ConvertedThis = convert(CommonFXSema); + APFixedPoint ConvertedOther = Other.convert(CommonFXSema); + llvm::APSInt ThisVal = ConvertedThis.getValue(); + llvm::APSInt OtherVal = ConvertedOther.getValue(); + bool Overflowed = false; + + llvm::APSInt Result; + if (CommonFXSema.isSaturated()) { +Result = CommonFXSema.isSigned() ? ThisVal.ssub_sat(OtherVal) + : ThisVal.usub_sat(OtherVal); + } else { +Result = ThisVal.isSigned() ? ThisVal.ssub_ov(OtherVal, Overflowed) +: ThisVal.usub_ov(OtherVal, Overflowed); + } + + if (Overflow) +*Overflow = Overflowed; + + return APFixedPoint(Result
[PATCH] D73185: [AST] Add fixed-point subtraction constant evaluation.
rjmccall accepted this revision. rjmccall added a comment. LGTM. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73185/new/ https://reviews.llvm.org/D73185 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D73185: [AST] Add fixed-point subtraction constant evaluation.
ebevhan updated this revision to Diff 239791. ebevhan added a comment. Move packing of FixedPointSemantics to a separate patch. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73185/new/ https://reviews.llvm.org/D73185 Files: clang/include/clang/Basic/FixedPoint.h clang/lib/AST/ExprConstant.cpp clang/lib/Basic/FixedPoint.cpp clang/test/Frontend/fixed_point_sub.c Index: clang/test/Frontend/fixed_point_sub.c === --- clang/test/Frontend/fixed_point_sub.c +++ clang/test/Frontend/fixed_point_sub.c @@ -1,6 +1,55 @@ // RUN: %clang_cc1 -ffixed-point -S -emit-llvm %s -o - | FileCheck %s --check-prefixes=CHECK,SIGNED // RUN: %clang_cc1 -ffixed-point -fpadding-on-unsigned-fixed-point -S -emit-llvm %s -o - | FileCheck %s --check-prefixes=CHECK,UNSIGNED +// Subtraction between different fixed point types +short _Accum sa_const = 1.0hk - 2.0hk; // CHECK-DAG: @sa_const = {{.*}}global i16 -128, align 2 +_Accum a_const = 1.0hk - 2.0k; // CHECK-DAG: @a_const = {{.*}}global i32 -32768, align 4 +long _Accum la_const = 1.0hk - 2.0lk; // CHECK-DAG: @la_const = {{.*}}global i64 -2147483648, align 8 +short _Accum sa_const2 = 0.5hr - 2.0hk; // CHECK-DAG: @sa_const2 = {{.*}}global i16 -192, align 2 +short _Accum sa_const3 = 0.5r - 2.0hk; // CHECK-DAG: @sa_const3 = {{.*}}global i16 -192, align 2 +short _Accum sa_const4 = 0.5lr - 2.0hk; // CHECK-DAG: @sa_const4 = {{.*}}global i16 -192, align 2 +short _Accum sa_const5 = 2.0hk - 0.5lr; // CHECK-DAG: @sa_const5 = {{.*}}global i16 192, align 2 + +// Unsigned subtraction +unsigned short _Accum usa_const = 3.0uhk - 2.0uhk; +// CHECK-SIGNED-DAG: @usa_const = {{.*}}global i16 768, align 2 +// CHECK-UNSIGNED-DAG: @usa_const = {{.*}}global i16 384, align 2 + +// Unsigned - signed +short _Accum sa_const6 = 1.0uhk - 2.0hk; +// CHECK-DAG: @sa_const6 = {{.*}}global i16 -128, align 2 + +// Subtraction with negative number +short _Accum sa_const7 = 0.5hr - (-2.0hk); +// CHECK-DAG: @sa_const7 = {{.*}}global i16 320, align 2 + +// Int subtraction +unsigned short _Accum usa_const2 = 2 - 0.5uhk; +// CHECK-SIGNED-DAG: @usa_const2 = {{.*}}global i16 640, align 2 +// CHECK-UNSIGNED-DAG: @usa_const2 = {{.*}}global i16 320, align 2 +short _Accum sa_const8 = 2 - (-0.5hk); // CHECK-DAG: @sa_const8 = {{.*}}global i16 320, align 2 +short _Accum sa_const9 = 257 - 2.0hk;// CHECK-DAG: @sa_const9 = {{.*}}global i16 32640, align 2 +long _Fract lf_const = 0.5lr - 1;// CHECK-DAG: @lf_const = {{.*}}global i32 -1073741824, align 4 + +// Saturated subtraction +_Sat short _Accum sat_sa_const = (_Sat short _Accum)128.0hk - (-128.0hk); +// CHECK-DAG: @sat_sa_const = {{.*}}global i16 32767, align 2 +_Sat unsigned short _Accum sat_usa_const = (_Sat unsigned short _Accum)128.0uhk - (-128.0uhk); +// CHECK-SIGNED-DAG: @sat_usa_const = {{.*}}global i16 65535, align 2 +// CHECK-UNSIGNED-DAG: @sat_usa_const = {{.*}}global i16 32767, align 2 +_Sat short _Accum sat_sa_const2 = (_Sat short _Accum)128.0hk - (-128); +// CHECK-DAG: @sat_sa_const2 = {{.*}}global i16 32767, align 2 +_Sat unsigned short _Accum sat_usa_const2 = (_Sat unsigned short _Accum)128.0uhk - (-128); +// CHECK-SIGNED-DAG: @sat_usa_const2 = {{.*}}global i16 65535, align 2 +// CHECK-UNSIGNED-DAG: @sat_usa_const2 = {{.*}}global i16 32767, align 2 +_Sat unsigned short _Accum sat_usa_const3 = (_Sat unsigned short _Accum)0.5uhk - 2; +// CHECK-DAG: @sat_usa_const3 = {{.*}}global i16 0, align 2 +_Sat short _Accum sat_sa_const3 = (_Sat short _Accum)-128.0hk - 128; +// CHECK-DAG: @sat_sa_const3 = {{.*}}global i16 -32768, align 2 +_Sat short _Accum sat_sa_const4 = (_Sat short _Accum)-150.0hk - 130.0lk; +// CHECK-DAG: @sat_sa_const4 = {{.*}}global i16 -32768, align 2 + + void SignedSubtraction() { // CHECK-LABEL: SignedSubtraction short _Accum sa; Index: clang/lib/Basic/FixedPoint.cpp === --- clang/lib/Basic/FixedPoint.cpp +++ clang/lib/Basic/FixedPoint.cpp @@ -173,6 +173,30 @@ return APFixedPoint(Result, CommonFXSema); } +APFixedPoint APFixedPoint::sub(const APFixedPoint &Other, + bool *Overflow) const { + auto CommonFXSema = Sema.getCommonSemantics(Other.getSemantics()); + APFixedPoint ConvertedThis = convert(CommonFXSema); + APFixedPoint ConvertedOther = Other.convert(CommonFXSema); + llvm::APSInt ThisVal = ConvertedThis.getValue(); + llvm::APSInt OtherVal = ConvertedOther.getValue(); + bool Overflowed = false; + + llvm::APSInt Result; + if (CommonFXSema.isSaturated()) { +Result = CommonFXSema.isSigned() ? ThisVal.ssub_sat(OtherVal) + : ThisVal.usub_sat(OtherVal); + } else { +Result = ThisVal.isSigned() ? ThisVal.ssub_ov(OtherVal, Overflowed) +: ThisVal.usub_ov(OtherVal, Overflowed); + } + + if (Overflow) +*Overfl
[PATCH] D73185: [AST] Add fixed-point subtraction constant evaluation.
ebevhan marked an inline comment as done. ebevhan added inline comments. Comment at: clang/include/clang/Basic/FixedPoint.h:82 + unsigned IsSaturated: 1; + unsigned HasUnsignedPadding : 1; }; rjmccall wrote: > These changes should probably be done in a separate patch. > > I don't have a problem with compressing this type, but I'm curious if there > was a concrete motivation. I can do it separately. There wasn't really a strong motivation other than that it could be packed better than it was. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73185/new/ https://reviews.llvm.org/D73185 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D73185: [AST] Add fixed-point subtraction constant evaluation.
rjmccall accepted this revision. rjmccall added a comment. This revision is now accepted and ready to land. LGTM other than the request to split the commit. Comment at: clang/include/clang/Basic/FixedPoint.h:82 + unsigned IsSaturated: 1; + unsigned HasUnsignedPadding : 1; }; These changes should probably be done in a separate patch. I don't have a problem with compressing this type, but I'm curious if there was a concrete motivation. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73185/new/ https://reviews.llvm.org/D73185 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D73185: [AST] Add fixed-point subtraction constant evaluation.
ebevhan created this revision. ebevhan added reviewers: rjmccall, leonardchan. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D73185 Files: clang/include/clang/Basic/FixedPoint.h clang/lib/AST/ExprConstant.cpp clang/lib/Basic/FixedPoint.cpp clang/test/Frontend/fixed_point_sub.c Index: clang/test/Frontend/fixed_point_sub.c === --- clang/test/Frontend/fixed_point_sub.c +++ clang/test/Frontend/fixed_point_sub.c @@ -1,6 +1,55 @@ // RUN: %clang_cc1 -ffixed-point -S -emit-llvm %s -o - | FileCheck %s --check-prefixes=CHECK,SIGNED // RUN: %clang_cc1 -ffixed-point -fpadding-on-unsigned-fixed-point -S -emit-llvm %s -o - | FileCheck %s --check-prefixes=CHECK,UNSIGNED +// Subtraction between different fixed point types +short _Accum sa_const = 1.0hk - 2.0hk; // CHECK-DAG: @sa_const = {{.*}}global i16 -128, align 2 +_Accum a_const = 1.0hk - 2.0k; // CHECK-DAG: @a_const = {{.*}}global i32 -32768, align 4 +long _Accum la_const = 1.0hk - 2.0lk; // CHECK-DAG: @la_const = {{.*}}global i64 -2147483648, align 8 +short _Accum sa_const2 = 0.5hr - 2.0hk; // CHECK-DAG: @sa_const2 = {{.*}}global i16 -192, align 2 +short _Accum sa_const3 = 0.5r - 2.0hk; // CHECK-DAG: @sa_const3 = {{.*}}global i16 -192, align 2 +short _Accum sa_const4 = 0.5lr - 2.0hk; // CHECK-DAG: @sa_const4 = {{.*}}global i16 -192, align 2 +short _Accum sa_const5 = 2.0hk - 0.5lr; // CHECK-DAG: @sa_const5 = {{.*}}global i16 192, align 2 + +// Unsigned subtraction +unsigned short _Accum usa_const = 3.0uhk - 2.0uhk; +// CHECK-SIGNED-DAG: @usa_const = {{.*}}global i16 768, align 2 +// CHECK-UNSIGNED-DAG: @usa_const = {{.*}}global i16 384, align 2 + +// Unsigned - signed +short _Accum sa_const6 = 1.0uhk - 2.0hk; +// CHECK-DAG: @sa_const6 = {{.*}}global i16 -128, align 2 + +// Subtraction with negative number +short _Accum sa_const7 = 0.5hr - (-2.0hk); +// CHECK-DAG: @sa_const7 = {{.*}}global i16 320, align 2 + +// Int subtraction +unsigned short _Accum usa_const2 = 2 - 0.5uhk; +// CHECK-SIGNED-DAG: @usa_const2 = {{.*}}global i16 640, align 2 +// CHECK-UNSIGNED-DAG: @usa_const2 = {{.*}}global i16 320, align 2 +short _Accum sa_const8 = 2 - (-0.5hk); // CHECK-DAG: @sa_const8 = {{.*}}global i16 320, align 2 +short _Accum sa_const9 = 257 - 2.0hk;// CHECK-DAG: @sa_const9 = {{.*}}global i16 32640, align 2 +long _Fract lf_const = 0.5lr - 1;// CHECK-DAG: @lf_const = {{.*}}global i32 -1073741824, align 4 + +// Saturated subtraction +_Sat short _Accum sat_sa_const = (_Sat short _Accum)128.0hk - (-128.0hk); +// CHECK-DAG: @sat_sa_const = {{.*}}global i16 32767, align 2 +_Sat unsigned short _Accum sat_usa_const = (_Sat unsigned short _Accum)128.0uhk - (-128.0uhk); +// CHECK-SIGNED-DAG: @sat_usa_const = {{.*}}global i16 65535, align 2 +// CHECK-UNSIGNED-DAG: @sat_usa_const = {{.*}}global i16 32767, align 2 +_Sat short _Accum sat_sa_const2 = (_Sat short _Accum)128.0hk - (-128); +// CHECK-DAG: @sat_sa_const2 = {{.*}}global i16 32767, align 2 +_Sat unsigned short _Accum sat_usa_const2 = (_Sat unsigned short _Accum)128.0uhk - (-128); +// CHECK-SIGNED-DAG: @sat_usa_const2 = {{.*}}global i16 65535, align 2 +// CHECK-UNSIGNED-DAG: @sat_usa_const2 = {{.*}}global i16 32767, align 2 +_Sat unsigned short _Accum sat_usa_const3 = (_Sat unsigned short _Accum)0.5uhk - 2; +// CHECK-DAG: @sat_usa_const3 = {{.*}}global i16 0, align 2 +_Sat short _Accum sat_sa_const3 = (_Sat short _Accum)-128.0hk - 128; +// CHECK-DAG: @sat_sa_const3 = {{.*}}global i16 -32768, align 2 +_Sat short _Accum sat_sa_const4 = (_Sat short _Accum)-150.0hk - 130.0lk; +// CHECK-DAG: @sat_sa_const4 = {{.*}}global i16 -32768, align 2 + + void SignedSubtraction() { // CHECK-LABEL: SignedSubtraction short _Accum sa; Index: clang/lib/Basic/FixedPoint.cpp === --- clang/lib/Basic/FixedPoint.cpp +++ clang/lib/Basic/FixedPoint.cpp @@ -173,6 +173,30 @@ return APFixedPoint(Result, CommonFXSema); } +APFixedPoint APFixedPoint::sub(const APFixedPoint &Other, + bool *Overflow) const { + auto CommonFXSema = Sema.getCommonSemantics(Other.getSemantics()); + APFixedPoint ConvertedThis = convert(CommonFXSema); + APFixedPoint ConvertedOther = Other.convert(CommonFXSema); + llvm::APSInt ThisVal = ConvertedThis.getValue(); + llvm::APSInt OtherVal = ConvertedOther.getValue(); + bool Overflowed = false; + + llvm::APSInt Result; + if (CommonFXSema.isSaturated()) { +Result = CommonFXSema.isSigned() ? ThisVal.ssub_sat(OtherVal) + : ThisVal.usub_sat(OtherVal); + } else { +Result = ThisVal.isSigned() ? ThisVal.ssub_ov(OtherVal, Overflowed) +: ThisVal.usub_ov(OtherVal, Overflowed); + } + + if (Overflow) +*Overflow = Overflowed; + + return APFixedPoint(Resu