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

Reply via email to