[PATCH] D55853: Ignore ConstantExpr in IgnoreParenNoopCasts

[Reid Kleckner via Phabricator via cfe-commits]

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