[PATCH] D83215: [AST][RecoveryExpr] Clarify the documentation of RecoveryExpr.

hokein abandoned this revision. hokein added a comment. closing this in favor of D83213 . Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D83215/new/ https://reviews.llvm.org/D83215

[PATCH] D83215: [AST][RecoveryExpr] Clarify the documentation of RecoveryExpr.

hokein added inline comments. Comment at: clang/lib/AST/ComputeDependence.cpp:498 ExprDependence clang::computeDependence(RecoveryExpr *E) { - // Mark the expression as value- and instantiation- dependent to reuse - // existing suppressions for dependent code, e.g. avoiding -

[PATCH] D83215: [AST][RecoveryExpr] Clarify the documentation of RecoveryExpr.

hokein updated this revision to Diff 276700. hokein marked 4 inline comments as done. hokein added a comment. refine the doc based on the review comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D83215/new/ https://reviews.llvm.org/D83215

[PATCH] D83215: [AST][RecoveryExpr] Clarify the documentation of RecoveryExpr.

sammccall added inline comments. Comment at: clang/include/clang/AST/Expr.h:6229 +/// +/// FIXME: RecoveryExpr is C++ only, make it work for C by supporting dependence +/// mechanism for C language in clang. this could be a little more specific (e.g. what does

[PATCH] D83215: [AST][RecoveryExpr] Clarify the documentation of RecoveryExpr.

hokein created this revision. hokein added a reviewer: sammccall. Herald added a project: clang. Fix some out-of-date doc/FIXMEs, and be clear about our plan. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D83215 Files: clang/include/clang/AST/Expr.h