[clang-tools-extra] [clang] [clangd] Prevent printing huge initializer lists in hover definitions (PR #79746)

Raoul Wols via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 29 02:38:28 PST 2024


================
@@ -1720,6 +1720,12 @@ void StmtPrinter::VisitInitListExpr(InitListExpr* Node) {
   OS << "{";
   for (unsigned i = 0, e = Node->getNumInits(); i != e; ++i) {
     if (i) OS << ", ";
+    // TODO: There is duplicated functionality in APValue::printPretty.
+    // Would be good to consolidate the two.
+    if (!Policy.EntireContentsOfLargeArray && i == 10) {
----------------
rwols wrote:

I was also kind of unsure about the usage of `EntirecontentsOfLargeArray` here. I feel like StmtPrinter should always output valid C++ code  but that's not possible using `EntireContentsOfLargeArray`.

https://github.com/llvm/llvm-project/pull/79746


More information about the cfe-commits mailing list