https://github.com/jurahul edited
https://github.com/llvm/llvm-project/pull/109220
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jurahul wrote:
@AaronBallman ping to comment on this again. Not only does this have files
missing, it also includes file names that do not exist anymore in the repo:
```
clang/docs/tools/clang-formatted-files.txt:llvm/include/llvm/DWP/DWPStringPool.h
clang/docs/tools/clang-formatted-files.txt:l
jurahul wrote:
@AaronBallman WDTY? The motivation of the change is that I see most of the
changes in this file are a result of folks adding or deleting files, so its one
less thing to take care of when doing so. Also the current list of files in
this file is incomplete, further proving that th
jurahul wrote:
May be. Even in that PR, the only reference to this file is:
```
CLEAN_FILE = os.path.join(CLANG_DIR, 'docs/tools/clang-formatted-files.txt')
cleanfiles = open(CLEAN_FILE, "wb")
```
So its overwritten. But yeah, let @mydeveloperday confirm.
https://github.com/llvm/llvm-project/
AaronBallman wrote:
I don't think this file is actually unused; it was committed with the original
PR adding the script: 5aca8bb963a43b62f37305f7eb07476631ea9f2a
(https://reviews.llvm.org/D111000 was the review)
I've added @mydeveloperday as a reviewer for confirmation though
https://github.c
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Rahul Joshi (jurahul)
Changes
Looks like this file is generated by the `generate_formatted_state.py` script
in the same directory. So it seems this was committed to the repo inadvertently.
---
Patch is 408.98 KiB, truncated to 20.00 KiB
https://github.com/jurahul ready_for_review
https://github.com/llvm/llvm-project/pull/109220
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul edited
https://github.com/llvm/llvm-project/pull/109220
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
