[clang-tools-extra] [clang-doc] Add helpers for Template config (PR #138062)

2025-05-23 Thread Paul Kirth via cfe-commits
ilovepi wrote: #141269 should fix the issue for now. I'll clean up the rest of the stack to avoid reintroducing this problem. https://github.com/llvm/llvm-project/pull/138062 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.o

[clang-tools-extra] [clang-doc] Add helpers for Template config (PR #138062)

2025-05-23 Thread Nico Weber via cfe-commits
nico wrote: > As an alternative, we could disable the unit tests for the new feature, since > its expected to be experimental for some time, and move the testing into lit > tests. That sounds like a good way forward to me :) https://github.com/llvm/llvm-project/pull/138062 ___

[clang-tools-extra] [clang-doc] Add helpers for Template config (PR #138062)

2025-05-23 Thread Paul Kirth via cfe-commits
ilovepi wrote: > Reading files from unit tests is a bit of an anti-pattern: It requires > putting the absolute path to the tests somewhere (you're doing this with a > config file here), and it means you easily run tests on a different machine > than the one you're building on. Could you restru

[clang-tools-extra] [clang-doc] Add helpers for Template config (PR #138062)

2025-05-23 Thread Nico Weber via cfe-commits
nico wrote: Reading files from unit tests is a bit of an anti-pattern: It requires putting the absolute path to the tests somewhere (you're doing this with a config file here), and it means you easily run tests on a different machine than the one you're building on. Could you restructure this

[clang-tools-extra] [clang-doc] Add helpers for Template config (PR #138062)

2025-05-22 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `clang-ppc64le-rhel` running on `ppc64le-clang-rhel-test` while building `clang-tools-extra` at step 7 "test-build-unified-tree-check-all". Full details are available at: https://lab.llvm.org/buildbot/#/builders/145/builds/720

[clang-tools-extra] [clang-doc] Add helpers for Template config (PR #138062)

2025-05-22 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `clang-ppc64le-linux-multistage` running on `ppc64le-clang-multistage-test` while building `clang-tools-extra` at step 5 "ninja check 1". Full details are available at: https://lab.llvm.org/buildbot/#/builders/76/builds/9814

[clang-tools-extra] [clang-doc] Add helpers for Template config (PR #138062)

2025-05-22 Thread Paul Kirth via cfe-commits
https://github.com/ilovepi closed https://github.com/llvm/llvm-project/pull/138062 ___ 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 helpers for Template config (PR #138062)

2025-05-22 Thread Paul Kirth via cfe-commits
ilovepi wrote: ### Merge activity * **May 22, 9:19 PM UTC**: A user started a stack merge that includes this pull request via [Graphite](https://app.graphite.dev/github/pr/llvm/llvm-project/138062). https://github.com/llvm/llvm-project/pull/138062

[clang-tools-extra] [clang-doc] Add helpers for Template config (PR #138062)

2025-05-22 Thread Paul Kirth via cfe-commits
https://github.com/ilovepi updated https://github.com/llvm/llvm-project/pull/138062 >From 86dc2be43374138ecea82f0266f8b8ae7ac17f4e Mon Sep 17 00:00:00 2001 From: Paul Kirth Date: Wed, 30 Apr 2025 08:10:20 -0700 Subject: [PATCH] [clang-doc] Add helpers for Template config This patch adds or fil

[clang-tools-extra] [clang-doc] Add helpers for Template config (PR #138062)

2025-05-20 Thread Daniel Thornburgh via cfe-commits
@@ -74,7 +75,51 @@ static std::unique_ptr NamespaceTemplate = nullptr; static std::unique_ptr RecordTemplate = nullptr; +static Error +setupTemplate(std::unique_ptr &Template, + StringRef TemplatePath, + std::vector> Partials) { + auto T = Mustache

[clang-tools-extra] [clang-doc] Add helpers for Template config (PR #138062)

2025-05-20 Thread Daniel Thornburgh via cfe-commits
https://github.com/mysterymath approved this pull request. https://github.com/llvm/llvm-project/pull/138062 ___ 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 helpers for Template config (PR #138062)

2025-05-20 Thread Daniel Thornburgh via cfe-commits
@@ -74,7 +75,51 @@ static std::unique_ptr NamespaceTemplate = nullptr; static std::unique_ptr RecordTemplate = nullptr; +static Error +setupTemplate(std::unique_ptr &Template, + StringRef TemplatePath, + std::vector> Partials) { + auto T = Mustache

[clang-tools-extra] [clang-doc] Add helpers for Template config (PR #138062)

2025-05-19 Thread Paul Kirth via cfe-commits
@@ -6,4 +6,6 @@ set(LLVM_LINK_COMPONENTS add_clang_library(clangDocSupport STATIC File.cpp - ) \ No newline at end of file + Utils.cpp + ) + ilovepi wrote: oh, CMake file ... 🤦 https://github.com/llvm/llvm-project/pull/138062 __

[clang-tools-extra] [clang-doc] Add helpers for Template config (PR #138062)

2025-05-19 Thread Paul Kirth via cfe-commits
@@ -0,0 +1,61 @@ +//===--===// +// +// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. +// See https://llvm.org/LICENSE.txt for license information. +// SPDX-License-Identifier: Apac

[clang-tools-extra] [clang-doc] Add helpers for Template config (PR #138062)

2025-05-19 Thread Paul Kirth via cfe-commits
@@ -6,4 +6,6 @@ set(LLVM_LINK_COMPONENTS add_clang_library(clangDocSupport STATIC File.cpp - ) \ No newline at end of file + Utils.cpp + ) + ilovepi wrote: one would hope `git calng-format would catch this...`. Will fix. https://github.com/llvm/llvm-pro

[clang-tools-extra] [clang-doc] Add helpers for Template config (PR #138062)

2025-05-19 Thread Paul Kirth via cfe-commits
@@ -74,7 +75,51 @@ static std::unique_ptr NamespaceTemplate = nullptr; static std::unique_ptr RecordTemplate = nullptr; +static Error +setupTemplate(std::unique_ptr &Template, + StringRef TemplatePath, + std::vector> Partials) { + auto T = Mustache

[clang-tools-extra] [clang-doc] Add helpers for Template config (PR #138062)

2025-05-17 Thread Erick Velez via cfe-commits
@@ -40,3 +51,4 @@ target_link_libraries(ClangDocTests clangDoc LLVMTestingSupport ) + evelez7 wrote: extra newline https://github.com/llvm/llvm-project/pull/138062 ___ cfe-commits mailing list cfe-commits@list

[clang-tools-extra] [clang-doc] Add helpers for Template config (PR #138062)

2025-05-16 Thread Erick Velez via cfe-commits
https://github.com/evelez7 approved this pull request. https://github.com/llvm/llvm-project/pull/138062 ___ 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 helpers for Template config (PR #138062)

2025-05-16 Thread Erick Velez via cfe-commits
@@ -6,4 +6,6 @@ set(LLVM_LINK_COMPONENTS add_clang_library(clangDocSupport STATIC File.cpp - ) \ No newline at end of file + Utils.cpp + ) + evelez7 wrote: nit: extra newline https://github.com/llvm/llvm-project/pull/138062 _

[clang-tools-extra] [clang-doc] Add helpers for Template config (PR #138062)

2025-05-16 Thread Erick Velez via cfe-commits
@@ -0,0 +1,61 @@ +//===--===// +// +// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. +// See https://llvm.org/LICENSE.txt for license information. +// SPDX-License-Identifier: Apac

[clang-tools-extra] [clang-doc] Add helpers for Template config (PR #138062)

2025-05-16 Thread Erick Velez via cfe-commits
@@ -74,7 +75,51 @@ static std::unique_ptr NamespaceTemplate = nullptr; static std::unique_ptr RecordTemplate = nullptr; +static Error +setupTemplate(std::unique_ptr &Template, + StringRef TemplatePath, + std::vector> Partials) { + auto T = Mustache

[clang-tools-extra] [clang-doc] Add helpers for Template config (PR #138062)

2025-05-16 Thread Paul Kirth via cfe-commits
https://github.com/ilovepi updated https://github.com/llvm/llvm-project/pull/138062 >From 3d870717fdc1a35f3fc28e7fa74b763362f98c19 Mon Sep 17 00:00:00 2001 From: Paul Kirth Date: Wed, 30 Apr 2025 08:10:20 -0700 Subject: [PATCH] [clang-doc] Add helpers for Template config This patch adds or fil

[clang-tools-extra] [clang-doc] Add helpers for Template config (PR #138062)

2025-05-16 Thread Paul Kirth via cfe-commits
https://github.com/ilovepi edited https://github.com/llvm/llvm-project/pull/138062 ___ 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 helpers for Template config (PR #138062)

2025-05-16 Thread Paul Kirth via cfe-commits
https://github.com/ilovepi updated https://github.com/llvm/llvm-project/pull/138062 >From 442fe2fd1353c540bc0ec39ec913e7e414def6bc Mon Sep 17 00:00:00 2001 From: Paul Kirth Date: Wed, 30 Apr 2025 08:10:20 -0700 Subject: [PATCH] [clang-doc] Add helpers for Template config This patch adds or fil