cpplearner added a comment. In D65050#1596514 <https://reviews.llvm.org/D65050#1596514>, @efriedma wrote:
> Is this the only place where a compound type can contain a PackExpansionType? Yes AFAIK. No other place can contain a list or a pack expansion. The closest thing is dynamic exception specification (which isn't part of the type), and it seems to be handled properly. > The code could probably use a brief comment explaining why we need to check > this explicitly, even though it isn't listed in [temp.dep.type]. Done. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65050/new/ https://reviews.llvm.org/D65050 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits