[PATCH] D132997: [clang][Interp] Handle DeclRefExpr of reference types

Timm Bäder via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 1 01:17:04 PDT 2022


tbaeder added a comment.

In D132997#3763094 <https://reviews.llvm.org/D132997#3763094>, @shafik wrote:

> I think like @aaron.ballman was saying in another PR you should aim for as complete set of tests as possible even if you have to comment one that can't work yet. For example cases that would involve `MemberExpr` or `UsingShadow` for example as well as the cases you mentioned not implemented in your description.

I get it for the `MaterializeTemporaryExpr`s, but I don't understand why I would add tests using `MemberExpr` now and not when I work on record types (and references have already landed).


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D132997/new/

https://reviews.llvm.org/D132997



More information about the cfe-commits mailing list