[PATCH] D55853: Ignore ConstantExpr in IgnoreParenNoopCasts

David Majnemer via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 18 14:17:36 PST 2018


majnemer added inline comments.


================
Comment at: clang/lib/AST/Expr.cpp:2722-2725
+    if (ConstantExpr *CE = dyn_cast<ConstantExpr>(E)) {
+      E = CE->getSubExpr();
+      continue;
+    }
----------------
Just curious, why not go even further and use FullExpr rather than ConstantExpr?


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

https://reviews.llvm.org/D55853





More information about the cfe-commits mailing list