================ @@ -20008,6 +20008,87 @@ bool Sema::IsValueInFlagEnum(const EnumDecl *ED, const llvm::APInt &Val, return !(FlagMask & Val) || (AllowMask && !(FlagMask & ~Val)); } +bool Sema::ComputeBestEnumProperties(ASTContext &Context, EnumDecl *Enum, + bool is_cpp, bool isPacked, ---------------- kuilpd wrote:
> > EnumDecl can fill it's own BestType and BestPromotionType fields using this > > function. > > I'm afraid it might result in EnumDecl taking more space and I don't think > we'd like that. I'm sorry, I'm a bit confused here, what kind of space do you mean? I suggest adding only one method to the class, nothing else. https://github.com/llvm/llvm-project/pull/120965 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits