[clang-tools-extra] [clang-doc] add namespace references to VarInfo (PR #146964)

2025-07-03 Thread Erick Velez via cfe-commits
https://github.com/evelez7 closed https://github.com/llvm/llvm-project/pull/146964 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-doc] add namespace references to VarInfo (PR #146964)

2025-07-03 Thread Erick Velez via cfe-commits
evelez7 wrote: ### Merge activity * **Jul 3, 10:49 PM UTC**: A user started a stack merge that includes this pull request via [Graphite](https://app.graphite.dev/github/pr/llvm/llvm-project/146964). https://github.com/llvm/llvm-project/pull/146964

[clang-tools-extra] [clang-doc] add namespace references to VarInfo (PR #146964)

2025-07-03 Thread Erick Velez via cfe-commits
https://github.com/evelez7 updated https://github.com/llvm/llvm-project/pull/146964 >From f505202482e624dd4f7950cded2768889f1ee2d8 Mon Sep 17 00:00:00 2001 From: Erick Velez Date: Wed, 2 Jul 2025 21:49:37 -0700 Subject: [PATCH] [clang-doc] add namespace references to VarInfo VarInfo was missin

[clang-tools-extra] [clang-doc] add namespace references to VarInfo (PR #146964)

2025-07-03 Thread Petr Hosek via cfe-commits
https://github.com/petrhosek approved this pull request. https://github.com/llvm/llvm-project/pull/146964 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-doc] add namespace references to VarInfo (PR #146964)

2025-07-03 Thread Petr Hosek via cfe-commits
@@ -0,0 +1,36 @@ +// RUN: rm -rf %t && mkdir -p %t +// RUN: clang-doc --output=%t --format=json --executor=standalone %s +// RUN: FileCheck %s < %t/nested/index.json --check-prefix=NESTED +// RUN: FileCheck %s < %t/nested/inner/index.json --check-prefix=INNER + +namespace nested {

[clang-tools-extra] [clang-doc] add namespace references to VarInfo (PR #146964)

2025-07-03 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-tools-extra Author: Erick Velez (evelez7) Changes VarInfo was missing its addReference specialization. This causes errors when a VarInfo is inside a namespace that isn't the global namespace. --- Full diff: https://github.com/llvm/llvm-project/pul

[clang-tools-extra] [clang-doc] add namespace references to VarInfo (PR #146964)

2025-07-03 Thread Erick Velez via cfe-commits
https://github.com/evelez7 ready_for_review https://github.com/llvm/llvm-project/pull/146964 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-doc] add namespace references to VarInfo (PR #146964)

2025-07-03 Thread Erick Velez via cfe-commits
evelez7 wrote: * **#146964** https://app.graphite.dev/github/pr/llvm/llvm-project/146964?utm_source=stack-comment-icon"; target="_blank">https://static.graphite.dev/graphite-32x32-black.png"; alt="Graphite" width="10px" height="10px"/> 👈 https://app.graphite.dev/github/pr/llvm/llvm-project/1469

[clang-tools-extra] [clang-doc] add namespace references to VarInfo (PR #146964)

2025-07-03 Thread Erick Velez via cfe-commits
https://github.com/evelez7 created https://github.com/llvm/llvm-project/pull/146964 VarInfo was missing its addReference specialization. This causes errors when a VarInfo is inside a namespace that isn't the global namespace. >From c686da5544239f10aa71c15f510d07a7268a629f Mon Sep 17 00:00:00 20