[PATCH] D73257: [AST] Compress the FixedPointSemantics type better.
This revision was automatically updated to reflect the committed changes. Closed by commit rGd5d0d8eb7d09: [AST] Compress the FixedPointSemantics type better. (authored by ebevhan). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73257/new/ https://reviews.llvm.org/D73257 Files: clang/include/clang/Basic/FixedPoint.h Index: clang/include/clang/Basic/FixedPoint.h === --- clang/include/clang/Basic/FixedPoint.h +++ clang/include/clang/Basic/FixedPoint.h @@ -75,11 +75,11 @@ } private: - unsigned Width; - unsigned Scale; - bool IsSigned; - bool IsSaturated; - bool HasUnsignedPadding; + unsigned Width : 16; + unsigned Scale : 13; + unsigned IsSigned : 1; + unsigned IsSaturated: 1; + unsigned HasUnsignedPadding : 1; }; /// The APFixedPoint class works similarly to APInt/APSInt in that it is a Index: clang/include/clang/Basic/FixedPoint.h === --- clang/include/clang/Basic/FixedPoint.h +++ clang/include/clang/Basic/FixedPoint.h @@ -75,11 +75,11 @@ } private: - unsigned Width; - unsigned Scale; - bool IsSigned; - bool IsSaturated; - bool HasUnsignedPadding; + unsigned Width : 16; + unsigned Scale : 13; + unsigned IsSigned : 1; + unsigned IsSaturated: 1; + unsigned HasUnsignedPadding : 1; }; /// The APFixedPoint class works similarly to APInt/APSInt in that it is a ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D73257: [AST] Compress the FixedPointSemantics type better.
ebevhan updated this revision to Diff 255974. ebevhan added a comment. Rebased. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73257/new/ https://reviews.llvm.org/D73257 Files: clang/include/clang/Basic/FixedPoint.h Index: clang/include/clang/Basic/FixedPoint.h === --- clang/include/clang/Basic/FixedPoint.h +++ clang/include/clang/Basic/FixedPoint.h @@ -75,11 +75,11 @@ } private: - unsigned Width; - unsigned Scale; - bool IsSigned; - bool IsSaturated; - bool HasUnsignedPadding; + unsigned Width : 16; + unsigned Scale : 13; + unsigned IsSigned : 1; + unsigned IsSaturated: 1; + unsigned HasUnsignedPadding : 1; }; /// The APFixedPoint class works similarly to APInt/APSInt in that it is a Index: clang/include/clang/Basic/FixedPoint.h === --- clang/include/clang/Basic/FixedPoint.h +++ clang/include/clang/Basic/FixedPoint.h @@ -75,11 +75,11 @@ } private: - unsigned Width; - unsigned Scale; - bool IsSigned; - bool IsSaturated; - bool HasUnsignedPadding; + unsigned Width : 16; + unsigned Scale : 13; + unsigned IsSigned : 1; + unsigned IsSaturated: 1; + unsigned HasUnsignedPadding : 1; }; /// The APFixedPoint class works similarly to APInt/APSInt in that it is a ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D73257: [AST] Compress the FixedPointSemantics type better.
rjmccall accepted this revision. rjmccall added a comment. This revision is now accepted and ready to land. Thanks, LGTM. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73257/new/ https://reviews.llvm.org/D73257 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D73257: [AST] Compress the FixedPointSemantics type better.
ebevhan updated this revision to Diff 239796. ebevhan added a comment. Rebased. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73257/new/ https://reviews.llvm.org/D73257 Files: clang/include/clang/Basic/FixedPoint.h Index: clang/include/clang/Basic/FixedPoint.h === --- clang/include/clang/Basic/FixedPoint.h +++ clang/include/clang/Basic/FixedPoint.h @@ -75,11 +75,11 @@ } private: - unsigned Width; - unsigned Scale; - bool IsSigned; - bool IsSaturated; - bool HasUnsignedPadding; + unsigned Width : 16; + unsigned Scale : 13; + unsigned IsSigned : 1; + unsigned IsSaturated: 1; + unsigned HasUnsignedPadding : 1; }; /// The APFixedPoint class works similarly to APInt/APSInt in that it is a Index: clang/include/clang/Basic/FixedPoint.h === --- clang/include/clang/Basic/FixedPoint.h +++ clang/include/clang/Basic/FixedPoint.h @@ -75,11 +75,11 @@ } private: - unsigned Width; - unsigned Scale; - bool IsSigned; - bool IsSaturated; - bool HasUnsignedPadding; + unsigned Width : 16; + unsigned Scale : 13; + unsigned IsSigned : 1; + unsigned IsSaturated: 1; + unsigned HasUnsignedPadding : 1; }; /// The APFixedPoint class works similarly to APInt/APSInt in that it is a ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D73257: [AST] Compress the FixedPointSemantics type better.
ebevhan created this revision. ebevhan added reviewers: rjmccall, leonardchan, bjope. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D73257 Files: clang/include/clang/Basic/FixedPoint.h Index: clang/include/clang/Basic/FixedPoint.h === --- clang/include/clang/Basic/FixedPoint.h +++ clang/include/clang/Basic/FixedPoint.h @@ -75,11 +75,11 @@ } private: - unsigned Width; - unsigned Scale; - bool IsSigned; - bool IsSaturated; - bool HasUnsignedPadding; + unsigned Width : 16; + unsigned Scale : 13; + unsigned IsSigned : 1; + unsigned IsSaturated: 1; + unsigned HasUnsignedPadding : 1; }; /// The APFixedPoint class works similarly to APInt/APSInt in that it is a Index: clang/include/clang/Basic/FixedPoint.h === --- clang/include/clang/Basic/FixedPoint.h +++ clang/include/clang/Basic/FixedPoint.h @@ -75,11 +75,11 @@ } private: - unsigned Width; - unsigned Scale; - bool IsSigned; - bool IsSaturated; - bool HasUnsignedPadding; + unsigned Width : 16; + unsigned Scale : 13; + unsigned IsSigned : 1; + unsigned IsSaturated: 1; + unsigned HasUnsignedPadding : 1; }; /// The APFixedPoint class works similarly to APInt/APSInt in that it is a ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits