[clang-tools-extra] [clang-doc] refactor JSON for better Mustache compatibility (PR #149588)

Erick Velez via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 21 18:09:00 PDT 2025


================
@@ -777,7 +778,10 @@ static void populateSymbolInfo(SymbolInfo &I, const T *D, const FullComment *C,
     Mangler->mangleCXXVTable(CXXD, MangledStream);
   else
     MangledStream << D->getNameAsString();
-  I.MangledName = MangledName;
+  if (MangledName.size() > 255)
+    I.MangledName = llvm::toStringRef(llvm::toHex(I.USR));
+  else
+    I.MangledName = MangledName;
----------------
evelez7 wrote:

Windows max length was greatly increased in windows 10 but I don't know if we could opt-in to that.

https://learn.microsoft.com/en-us/windows/win32/fileio/maximum-file-path-limitation?tabs=registry

Here's a Wikipedia article on filesystems. Looks like path limits are not defined but file name limits are.

https://en.wikipedia.org/wiki/Comparison_of_file_systems#Limits

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


More information about the cfe-commits mailing list