Tyker added a comment.

> 2. Change the constant expression representation for a pointer or reference 
> for a materialized temporary to refer to the temporary declaration instead.

I analyzed change 2 in this list and this require changing a very significant 
number of users many of them being implicit. and as far as I understand with 
the issue with duplication of MaterializeTemporaryExpr is solved by this patch 
+ https://reviews.llvm.org/D70190. there can be multiple 
MaterializeTemporaryExpr but if they are equivalent they will all use the same 
LifetimeExtendedTemporaryDecl which contain all there internal state.


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

https://reviews.llvm.org/D69360



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

Reply via email to