================ @@ -1459,11 +1460,11 @@ static AccessResult CheckEffectiveAccess(Sema &S, llvm_unreachable("invalid access result"); } -static Sema::AccessResult CheckAccess(Sema &S, SourceLocation Loc, - AccessTarget &Entity) { +static SemaAccess::AccessResult CheckAccess(Sema &S, SourceLocation Loc, ---------------- erichkeane wrote:
i find myself wondering if this should just take `SemaAccess` instead of `sema`, with all the below churn. It seems to make more sense, even if it has to do an extraction of Sema in the function. https://github.com/llvm/llvm-project/pull/92674 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits