mibintc added a comment. In D72841#1911330 <https://reviews.llvm.org/D72841#1911330>, @rjmccall wrote:
> In D72841#1908084 <https://reviews.llvm.org/D72841#1908084>, @mibintc wrote: > > > @rjmccall suggested that I needed to remove FPOptions from the Stmt class > > since the sizeof assertion failed. I moved that information into the > > relevant expression nodes and fixed a few small things that I didn't like > > in the previous version. > > > You only need to do this for the expression nodes where it causes an overflow > in the size of the bit-field; I don't think you're overflowing the capacity > of UnaryOperatorBitfields, for example. I added unsigned FPFeatures : 14; to class UnaryOperatorBitfields but I encountered the same assertion failure in the Stmt constructor, In constructor ‘clang::Stmt::Stmt(clang::Stmt::StmtClass)’: /iusers/sandbox/pragma-ws/llvm-project/clang/include/clang/AST/Stmt.h:1095:5: error: static assertion failed: changing bitfields changed sizeof(Stmt) static_assert(sizeof(*this) <= 8 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72841/new/ https://reviews.llvm.org/D72841 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits