https://github.com/ojhunt requested changes to this pull request.

I'm not super happy with this kind of construction being represented as a 
functional cast expression - it is not correct from the AST PoV, and I'm not 
sure it is correct semantically either.

However we do this for list initializers already, and I'm not sure of the 
complexity of actually introducing a new list initializer specific node. If 
that is too heavy weight, I'd rather we add a 
CXXFunctionalCastExpr::CreateListInitializer(...) function to use instead of 
using a null SourceLocation for the parens as a side channel.

https://github.com/llvm/llvm-project/pull/228340
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to