[PATCH] D73185: [AST] Add fixed-point subtraction constant evaluation.
Bevin Hansson via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Jan 23 00:13:02 PST 2020
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
More information about the cfe-commits
mailing list