================ @@ -18,29 +18,42 @@ namespace clang::sema { -/// Describes an entity that is being assigned. -struct AssignedEntity { - // The left-hand side expression of the assignment. - Expr *LHS = nullptr; +struct CapturingEntity { ---------------- hokein wrote:
+1. I'd not change the existing the `AssignedEntity`, as this structure is designed for the assignment case. It looks like for the capture_by case, we only need the `Expression`, I think we could use `Expr*` directly in the function parameter, e.g. `void checkCaptureLifetime(Sema &SemaRef, const Expr* Entity, Expr *Captured);`. https://github.com/llvm/llvm-project/pull/111499 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits