kuilpd wrote: The code needs to be called from LLDB as well (#115005), so this is an NFC patch to move it to `ASTContext.h`, plus this cleans up a bit the code in `Sema::ActOnEnumBody`. `isRepresentableIntegerValue` also had to be moved to `ASTContext.h`, but since it requires `ASTContext` as a first argument anyway, it seems like it's a better location for this function anyway. The only question is that `computeEnumBits` has a raw template in order to function with different collections that Sema and LLDB use, so the function has to be defined in the header. Can this be a problem?
https://github.com/llvm/llvm-project/pull/126096 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
