jdoerfert accepted this revision. jdoerfert added a comment. This revision is now accepted and ready to land.
LGTM, some nits ================ Comment at: clang/lib/CodeGen/CGStmt.cpp:801 + } else if (LanguageRequiresProgress()) + LoopMustProgress = true; + ---------------- Nit: Add braces here. ================ Comment at: clang/lib/CodeGen/CGStmt.cpp:894 + } else if (LanguageRequiresProgress()) + LoopMustProgress = true; + ---------------- Same as above. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86841/new/ https://reviews.llvm.org/D86841 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits