From e2369fd197d9ed9916bf78b2c8f6d7b8e0d66691 Mon Sep 17 00:00:00 2001 From: Reid Kleckner Date: Wed, 13 Nov 2019 11:25:42 -0800 Subject: [PATCH] [clang-include-fixer] Skip .rc files when finding symbols Summary: For some reason CMake includes entries for .rc files, but find-all-symbols handles them improperly. See PR43993 Reviewers: sammccall, bkramer Subscribers: cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D70196 --- .../find-all-symbols/tool/run-find-all-symbols.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/clang-tools-extra/clang-include-fixer/find-all-symbols/tool/run-find-all-symbols.py b/clang-tools-extra/clang-include-fixer/find-all-symbols/tool/run-find-all-symbols.py index 5e9dde723435..02b100bd849b 100755 --- a/clang-tools-extra/clang-include-fixer/find-all-symbols/tool/run-find-all-symbols.py +++ b/clang-tools-extra/clang-include-fixer/find-all-symbols/tool/run-find-all-symbols.py @@ -89,6 +89,9 @@ def main(): database = json.load(open(os.path.join(build_path, db_path))) files = [entry['file'] for entry in database] + # Filter out .rc files on Windows. CMake includes them for some reason. + files = [f for f in files if not f.endswith('.rc')] + max_task = args.j if max_task == 0: max_task = multiprocessing.cpu_count()