arphaman accepted this revision. arphaman added a comment. This revision is now accepted and ready to land.
LGTM ================ Comment at: lib/Index/USRGeneration.cpp:819 } + if (const auto *const AT = dyn_cast<ArrayType>(T)) { + Out << "{"; ---------------- Nit: I don't think you really need the 2nd const here and in the next if. https://reviews.llvm.org/D38643 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits