[PATCH] D22587: [ASTContext] Fix part of DR224 for nontype template arguments

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 21 08:49:02 PDT 2016


aaron.ballman added a comment.

Generally LGTM with a small nit.


================
Comment at: lib/AST/ASTContext.cpp:4456
@@ +4455,3 @@
+      // the template parameter and not an expression involving the template parameter.
+      auto *E = Arg.getAsExpr()->IgnoreImpCasts();
+      while(auto *DeclRef = dyn_cast_or_null<DeclRefExpr>(E)) {
----------------
`const auto *` (and propagate it to other declarations)?


https://reviews.llvm.org/D22587





More information about the cfe-commits mailing list