[clang] [clang-tools-extra] [llvm] Remove clang-rename (PR #108988)

2024-09-19 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `clang-ppc64-aix` running on `aix-ppc64` while building `clang-tools-extra,clang,llvm,utils` at step 3 "clean-build-dir". Full details are available at: https://lab.llvm.org/buildbot/#/builders/64/builds/1020 Here is the rel

[clang] [clang-tools-extra] [llvm] Remove clang-rename (PR #108988)

2024-09-19 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `lld-x86_64-win` running on `as-worker-93` while building `clang-tools-extra,clang,llvm,utils` at step 7 "test-build-unified-tree-check-all". Full details are available at: https://lab.llvm.org/buildbot/#/builders/146/builds/1

[clang] [clang-tools-extra] [llvm] Remove clang-rename (PR #108988)

2024-09-18 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `sanitizer-x86_64-linux-android` running on `sanitizer-buildbot-android` while building `clang-tools-extra,clang,llvm,utils` at step 2 "annotate". Full details are available at: https://lab.llvm.org/buildbot/#/builders/186/bui

[clang] [clang-tools-extra] [llvm] Remove clang-rename (PR #108988)

2024-09-18 Thread Aaron Ballman via cfe-commits
https://github.com/AaronBallman closed https://github.com/llvm/llvm-project/pull/108988 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-tools-extra] [llvm] Remove clang-rename (PR #108988)

2024-09-17 Thread Jordan Rupprecht via cfe-commits
rupprecht wrote: > AFAIK the bazel build scripts are externally automatically updated based on > the cmake scripts, so I don't think we should be touching them. Bazel files are not updated automatically. Updating them in the same commit is fine (but not required). GN files are updated by the

[clang] [clang-tools-extra] [llvm] Remove clang-rename (PR #108988)

2024-09-17 Thread Jordan Rupprecht via cfe-commits
https://github.com/rupprecht commented: Bazel changes LGTM. Updating them is totally not necessary, but appreciated if you do. Thanks! https://github.com/llvm/llvm-project/pull/108988 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lis

[clang] [clang-tools-extra] [llvm] Remove clang-rename (PR #108988)

2024-09-17 Thread Matheus Izvekov via cfe-commits
mizvekov wrote: AFAIK the bazel build scripts are externally automatically updated based on the cmake scripts, so I don't think we should be touching them. https://github.com/llvm/llvm-project/pull/108988 ___ cfe-commits mailing list cfe-commits@lists

[clang] [clang-tools-extra] [llvm] Remove clang-rename (PR #108988)

2024-09-17 Thread Kirill Bobyrev via cfe-commits
https://github.com/kirillbobyrev approved this pull request. I agree, clang-rename should probably be removed. I haven't touched it in years and its usability has dropped over the last few years after I re-implemented the most useful functionality in clangd. https://github.com/llvm/llvm-projec

[clang] [clang-tools-extra] [llvm] Remove clang-rename (PR #108988)

2024-09-17 Thread Aaron Ballman via cfe-commits
AaronBallman wrote: > Also these bazel build scripts > > ``` > llvm/utils/gn/secondary/clang-tools-extra/test/BUILD.gn: > "//clang/tools/clang-rename", > llvm/utils/gn/secondary/clang/test/BUILD.gn:"//clang/tools/clang-rename", > llvm/utils/gn/secondary/clang/tools/clang-rename/BUILD.gn: