[clang] 1fbbf92 - [perf-training] Check extension in findFilesWithExtension
Amir Ayupov via cfe-commits
cfe-commits at lists.llvm.org
Tue Jan 10 11:46:56 PST 2023
Author: Amir Ayupov
Date: 2023-01-10T11:46:51-08:00
New Revision: 1fbbf92e4fda3c7a3be1c02e1f7240135557846d
URL: https://github.com/llvm/llvm-project/commit/1fbbf92e4fda3c7a3be1c02e1f7240135557846d
DIFF: https://github.com/llvm/llvm-project/commit/1fbbf92e4fda3c7a3be1c02e1f7240135557846d.diff
LOG: [perf-training] Check extension in findFilesWithExtension
`findFilesWithExtension` helper checks for `endswith(extension)` instead of
exactly matching the file extension. This causes it to match unrelated files,
for example, `.profdata` files while matching `.fdata` files:
http://157.230.108.44:8011/#/builders/56/builds/247
```
Merging data from /worker/worker/bolt-x86_64-ubuntu-clang-bolt-gcc/build/tools/clang/prof.fdata.1124569.fdata...
Merging data from /worker/worker/bolt-x86_64-ubuntu-clang-bolt-gcc/build/tools/clang/test/Frontend/Output/optimization-remark-with-hotness-new-pm.c.tmp.profdata...
```
Reviewed By: phosek
Differential Revision: https://reviews.llvm.org/D141342
Added:
Modified:
clang/utils/perf-training/perf-helper.py
Removed:
################################################################################
diff --git a/clang/utils/perf-training/perf-helper.py b/clang/utils/perf-training/perf-helper.py
index c6a815e65473..d68ab3c72806 100644
--- a/clang/utils/perf-training/perf-helper.py
+++ b/clang/utils/perf-training/perf-helper.py
@@ -23,7 +23,7 @@ def findFilesWithExtension(path, extension):
filenames = []
for root, dirs, files in os.walk(path):
for filename in files:
- if filename.endswith(extension):
+ if os.path.splitext(filename)[1] == extension:
filenames.append(os.path.join(root, filename))
return filenames
More information about the cfe-commits
mailing list