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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to