[PATCH] D134020: [clang][Interp] Handle enums

Fri Sep 16 06:17:53 PDT 2022

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