tbaeder added a comment. In D134020#3795091 <https://reviews.llvm.org/D134020#3795091>, @erichkeane wrote:
> I can't come up with a way to get a reference to one of these, so the logic > in the rest of the DeclRefExpr handling is a little MORE awkward looking now. > I think I'm OK with this, but I want to give a few other folks a chance to > look. Ah right, I could to through that code path as well and not immediately return. I actually tested references but the AST expresses that through a `MaterializeTemporaryExpr`, which don't work yet. In D134020#3795092 <https://reviews.llvm.org/D134020#3795092>, @erichkeane wrote: > Also note, the CI test is from your test! Yup, that needs one of the other open patches. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134020/new/ https://reviews.llvm.org/D134020 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
