[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-10 Thread via cfe-commits
https://github.com/PeterChou1 updated https://github.com/llvm/llvm-project/pull/134298 >From fa35468f673ace035036a1c15d2d6ab756c2581e Mon Sep 17 00:00:00 2001 From: PeterChou1 Date: Thu, 3 Apr 2025 15:59:34 -0400 Subject: [PATCH 1/6] factor out file helpers --- clang-tools-extra/clang-doc/CMa

[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-10 Thread Martin Storsjö via cfe-commits
mstorsjo wrote: This change broke compiling with `-DLLVM_LINK_LLVM_DYLIB=ON`, as seen in the buildbot log above. Will revert later today if we don't have a fix. https://github.com/llvm/llvm-project/pull/134298 ___ cfe-commits mailing list cfe-commits@

[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-09 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `llvm-x86_64-debian-dylib` running on `gribozavr4` while building `clang-tools-extra` at step 5 "build-unified-tree". Full details are available at: https://lab.llvm.org/buildbot/#/builders/60/builds/24310 Here is the releva

[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-09 Thread Daniel Thornburgh via cfe-commits
https://github.com/mysterymath approved this pull request. https://github.com/llvm/llvm-project/pull/134298 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-09 Thread via cfe-commits
https://github.com/PeterChou1 closed https://github.com/llvm/llvm-project/pull/134298 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-07 Thread Paul Kirth via cfe-commits
@@ -0,0 +1,72 @@ +//===-- FileHelpersClangDoc.cpp - File Helpers ---*- C++-*-===// ilovepi wrote: Apparently, we don't use that line anymore. https://llvm.org/docs/CodingStandards.html#file-headers Was brought up here when I saw this the first

[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-07 Thread Paul Kirth via cfe-commits
https://github.com/ilovepi edited https://github.com/llvm/llvm-project/pull/134298 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-07 Thread Daniel Thornburgh via cfe-commits
@@ -0,0 +1,72 @@ +//===-- FileHelpersClangDoc.cpp - File Helpers ---*- C++-*-===// mysterymath wrote: The filename in this comment should match the actual filename. https://github.com/llvm/llvm-project/pull/134298 ___

[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-07 Thread Paul Kirth via cfe-commits
https://github.com/ilovepi edited https://github.com/llvm/llvm-project/pull/134298 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-05 Thread via cfe-commits
https://github.com/PeterChou1 updated https://github.com/llvm/llvm-project/pull/134298 >From fa35468f673ace035036a1c15d2d6ab756c2581e Mon Sep 17 00:00:00 2001 From: PeterChou1 Date: Thu, 3 Apr 2025 15:59:34 -0400 Subject: [PATCH 1/3] factor out file helpers --- clang-tools-extra/clang-doc/CMa

[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-04 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-tools-extra Author: None (PeterChou1) Changes Split from https://github.com/llvm/llvm-project/pull/133161 refactor the code to extract file helpers used in HTML generators for use in other generators for clang-doc --- Full diff: https://github.c

[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-03 Thread Paul Kirth via cfe-commits
https://github.com/ilovepi edited https://github.com/llvm/llvm-project/pull/134298 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-03 Thread Paul Kirth via cfe-commits
@@ -0,0 +1,25 @@ +//===-- File.h --- File Helpers ---*- C++-*-===// +// +// 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: Apache-2.0 WITH LLV

[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-03 Thread Paul Kirth via cfe-commits
https://github.com/ilovepi approved this pull request. Since this just moves common code I think this is fine for now. We should revisit some of the interfaces though, especially the ones that return SmallString<128>. I think there is also a Path API that handles relative paths, so we can per

[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-03 Thread Paul Kirth via cfe-commits
@@ -0,0 +1,25 @@ +//===-- File.h --- File Helpers ---*- C++-*-===// +// +// 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: Apache-2.0 WITH LLV

[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-03 Thread via cfe-commits
https://github.com/PeterChou1 updated https://github.com/llvm/llvm-project/pull/134298 >From fa35468f673ace035036a1c15d2d6ab756c2581e Mon Sep 17 00:00:00 2001 From: PeterChou1 Date: Thu, 3 Apr 2025 15:59:34 -0400 Subject: [PATCH 1/4] factor out file helpers --- clang-tools-extra/clang-doc/CMa

[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-03 Thread via cfe-commits
https://github.com/PeterChou1 updated https://github.com/llvm/llvm-project/pull/134298 >From fa35468f673ace035036a1c15d2d6ab756c2581e Mon Sep 17 00:00:00 2001 From: PeterChou1 Date: Thu, 3 Apr 2025 15:59:34 -0400 Subject: [PATCH 1/5] factor out file helpers --- clang-tools-extra/clang-doc/CMa

[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-03 Thread via cfe-commits
github-actions[bot] wrote: :warning: C/C++ code formatter, clang-format found issues in your code. :warning: You can test this locally with the following command: ``bash git-clang-format --diff HEAD~1 HEAD --extensions cpp,h -- clang-tools-extra/clang-doc/support/File.cpp clang

[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-03 Thread via cfe-commits
https://github.com/PeterChou1 updated https://github.com/llvm/llvm-project/pull/134298 >From fa35468f673ace035036a1c15d2d6ab756c2581e Mon Sep 17 00:00:00 2001 From: PeterChou1 Date: Thu, 3 Apr 2025 15:59:34 -0400 Subject: [PATCH 1/2] factor out file helpers --- clang-tools-extra/clang-doc/CMa

[clang-tools-extra] [clang-doc][NFC] refactor out file helpers (PR #134298)

2025-04-03 Thread via cfe-commits
https://github.com/PeterChou1 created https://github.com/llvm/llvm-project/pull/134298 Split from https://github.com/llvm/llvm-project/pull/133161 refactor the code to extract file helpers used in HTML generators for use in other generators for clang-doc >From fa35468f673ace035036a1c15d2d6ab7