Bug 307912
| Summary: | Static analyzer report processing is slow due to inefficient find+mv command | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | David Kilzer (:ddkilzer) <ddkilzer> |
| Component: | Tools / Tests | Assignee: | David Kilzer (:ddkilzer) <ddkilzer> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
David Kilzer (:ddkilzer)
The `analyze` target in `OpenSource/Makefile.shared` uses an inefficient `find -exec mv {} \;` command that spawns a new `mv` process for each report file found. With hundreds of report-*.html files generated during static analysis, this creates significant performance overhead.
The current implementation at line 203:
```
find $$dir -name 'report-*.html' -exec mv {} $$dir/StaticAnalyzerReports/ \;;
```
This spawns one `mv` process per file instead of batching files together, which is particularly slow when processing large numbers of static analyzer reports.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/170399490>
David Kilzer (:ddkilzer)
Pull request: https://github.com/WebKit/WebKit/pull/58730
EWS
Committed 307589@main (478fb33f559a): <https://commits.webkit.org/307589@main>
Reviewed commits have been landed. Closing PR #58730 and removing active labels.