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

Reply via email to