diff options
| author | Alex Richardson <Alexander.Richardson@cl.cam.ac.uk> | 2020-10-27 10:07:27 +0000 |
|---|---|---|
| committer | Alex Richardson <Alexander.Richardson@cl.cam.ac.uk> | 2020-10-27 12:16:46 +0000 |
| commit | 89031cffd1de0ce70106cda1bf5188f7a11726a2 (patch) | |
| tree | d1975c1dbc35784d36d3a96486b5250e6cb594fe /llvm/lib/CodeGen/LiveDebugValues/InstrRefBasedImpl.cpp | |
| parent | 850325348ae82cd5e26ea9edfd04219d0fbe7828 (diff) | |
| download | llvm-89031cffd1de0ce70106cda1bf5188f7a11726a2.zip llvm-89031cffd1de0ce70106cda1bf5188f7a11726a2.tar.gz llvm-89031cffd1de0ce70106cda1bf5188f7a11726a2.tar.bz2 | |
Fix sancov.py when objdump is llvm-objdump
The sanitizer-coverage.cpp test case was always failing for me. It turns
out the reason for this is that I was building with
-DLLVM_INSTALL_BINUTILS_SYMLINKS=ON and sancov.py's grep regex does not
handle llvm-objdump's disassembly format (hex immediates have a leading "0x").
While touching those lines also change them to use raw string literals since
invalid escape sequnces will become an error in future python versions.
Also simplify the code by using subprocess.check_output() instead of Popen().
This also works with python2.
Fixes https://bugs.llvm.org/show_bug.cgi?id=44504
Reviewed By: #sanitizers, vitalybuka
Differential Revision: https://reviews.llvm.org/D89648
Diffstat (limited to 'llvm/lib/CodeGen/LiveDebugValues/InstrRefBasedImpl.cpp')
0 files changed, 0 insertions, 0 deletions