[PATCH] D108482: [Clang] Fix instantiation of OpaqueValueExprs (Bug #45964)

[Jason Rice via Phabricator via cfe-commits]

ricejasonf marked an inline comment as done.
ricejasonf added a comment.

In D108482#3047141 <https://reviews.llvm.org/D108482#3047141>, @rjmccall wrote:

> I assume we rebuild an AILE and OVE when we process the initializer and see it's a structured binding?  In that case, this should be fine.

Yes, the original problem was that the old OVE was being nested in the new one resulting in AILE->OVE->OVE. CodeGen was triggering an assert as it is apparently not equipped to handle multiple OVEs simultaneously or at least not when they are directly nested like that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108482