================ @@ -57,6 +57,7 @@ mlir::LogicalResult CIRGenFunction::emitStmt(const Stmt *s, switch (s->getStmtClass()) { case Stmt::BreakStmtClass: + case Stmt::NullStmtClass: ---------------- andykaylor wrote:
I just looked at classic codegen. It puts NoStmtClass (along with CXXCatchStmtClass, SEHExceptStmtClass, SEHFinallyStmtClass, and MSDependentExistsStmtClass) in a different group just above this that leads to `llvm_unreachable("invalid statement class to emit generically");` rather than `llvm_unreachable("should have emitted these statements as simple");` as we have here. I can still do that here if you want, but it's currently in the group below that goes to an NYI error. https://github.com/llvm/llvm-project/pull/134889 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits