rnk marked an inline comment as done.
rnk added inline comments.
================
Comment at: clang/lib/AST/Expr.cpp:2722-2725
+ if (ConstantExpr *CE = dyn_cast<ConstantExpr>(E)) {
+ E = CE->getSubExpr();
+ continue;
+ }
----------------
majnemer wrote:
> Just curious, why not go even further and use FullExpr rather than
> ConstantExpr?
That would include MaterializeTemporaryExpr. Do you think we should consider
that a noop cast? You could make the argument that it is, but I was aiming to
match pre ConstantExpr behavior.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D55853/new/
https://reviews.llvm.org/D55853
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
