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
