[clang] [clang][Sema] Move computing enum width and type to a separate function (PR #120965)

[Ilia Kuklin via cfe-commits]

================
@@ -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