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
