diff options
| author | Amir Ayupov <aaupov@fb.com> | 2023-06-09 06:23:46 -0700 |
|---|---|---|
| committer | Amir Ayupov <aaupov@fb.com> | 2023-06-09 06:24:17 -0700 |
| commit | 5acac7db6e71db66d87987eaf5c91315d73ca086 (patch) | |
| tree | c4e8ae45cdfd4496627dfe605622cd7c4a152175 /clang/unittests/libclang/LibclangTest.cpp | |
| parent | c45695ef46570409717ca53820e1b0f0e59792a1 (diff) | |
| download | llvm-5acac7db6e71db66d87987eaf5c91315d73ca086.zip llvm-5acac7db6e71db66d87987eaf5c91315d73ca086.tar.gz llvm-5acac7db6e71db66d87987eaf5c91315d73ca086.tar.bz2 | |
[BOLT][NFCI] Use StringRef.split in launchPerfProcess
Use StringRef method instead of reimplementing the splitting.
Incidentally, it also fixes the duplicate printing of the command arguments:
```
PERF2BOLT: spawning perf job to read branch events
Launching perf: /usr/bin/perf script^@-F^@pid,ip,brstack -F^@pid,ip,brstack pid,ip,brstack -f -i
PERF2BOLT: spawning perf job to read mem events
Launching perf: /usr/bin/perf script^@-F^@pid,event,addr,ip -F^@pid,event,addr,ip pid,event,addr,ip -f -i
PERF2BOLT: spawning perf job to read process events
Launching perf: /usr/bin/perf script^@--show-mmap-events --show-mmap-events -f -i
PERF2BOLT: spawning perf job to read task events
Launching perf: /usr/bin/perf script^@--show-task-events --show-task-events -f -i
```
Fixes it to:
```
PERF2BOLT: spawning perf job to read branch events
Launching perf: /usr/bin/perf script -F pid,ip,brstack -f -i
PERF2BOLT: spawning perf job to read mem events
Launching perf: /usr/bin/perf script -F pid,event,addr,ip -f -i
PERF2BOLT: spawning perf job to read process events
Launching perf: /usr/bin/perf script --show-mmap-events -f -i
PERF2BOLT: spawning perf job to read task events
Launching perf: /usr/bin/perf script --show-task-events -f -i
```
Reviewed By: #bolt, rafauler
Differential Revision: https://reviews.llvm.org/D152483
Diffstat (limited to 'clang/unittests/libclang/LibclangTest.cpp')
0 files changed, 0 insertions, 0 deletions