hokein added a comment. In D120812#3354308 <https://reviews.llvm.org/D120812#3354308>, @sammccall wrote:
> Very nice! I thought we had considered and rejected this in the past, but I > think it was rather allowing lookup to find invalid decls in cases where it > doesn't today. Yeah, in the past (https://reviews.llvm.org/D76831), we tried to build the `DeclRefExpr` even for an invalid ref decl, it made the diagnostics much worse (as we don't know the type of the invalid decl, the fallback int type is not suitable for every cases). We use a different approach in this approach (build RecoveryExpr instead), a sad bit is that we are not able to retrieve the ref decl from the `RecoveryExpr`. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D120812/new/ https://reviews.llvm.org/D120812 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
