[all-commits] [llvm/llvm-project] 5725c8: [perf-training] Check extension in findFilesWithEx...

Amir Ayupov via All-commits all-commits at lists.llvm.org
Tue Jan 17 21:54:13 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 5725c8ddd46becff95629515b1f0a26af047aa33
      https://github.com/llvm/llvm-project/commit/5725c8ddd46becff95629515b1f0a26af047aa33
  Author: Amir Ayupov <aaupov at fb.com>
  Date:   2023-01-17 (Tue, 17 Jan 2023)

  Changed paths:
    M clang/utils/perf-training/perf-helper.py

  Log Message:
  -----------
  [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




More information about the All-commits mailing list