diff options
| author | Ilya Leoshkevich <iii@linux.ibm.com> | 2020-04-08 14:25:20 +0200 |
|---|---|---|
| committer | Ilya Leoshkevich <iii@linux.ibm.com> | 2020-04-08 14:25:59 +0200 |
| commit | aaba9a77615008624eb48fd333ac872c9e1d8883 (patch) | |
| tree | f6a9e2271deec53ad3f8a835f97e412f7a595f32 /llvm/lib/CodeGen/LiveDebugVariables.cpp | |
| parent | a533b03028fad4555e53b803197980735ccd68f9 (diff) | |
| download | llvm-aaba9a77615008624eb48fd333ac872c9e1d8883.zip llvm-aaba9a77615008624eb48fd333ac872c9e1d8883.tar.gz llvm-aaba9a77615008624eb48fd333ac872c9e1d8883.tar.bz2 | |
[compiler-rt] Don't use __libc_stack_end on ARM
Summary:
Commit b684c1a50f70 ("Add a `Symbolizer::GetEnvP()` method that allows
symbolizer implementations to customise the environment of the
symbolizer binary.") exposed a latent ARM issue, and that broke
http://lab.llvm.org:8011/builders/clang-cmake-thumbv7-full-sh
This coincided with breakage caused by my commit 5f5fb56c68e4
("[compiler-rt] Intercept the uname() function"), so I had to
investigate.
The issue is that GetArgsAndEnv does not work on ARM: there glibc's
_start overwrites argc value stored at __libc_start_end, breaking the
existing argv/envp parsing logic.
Fix by inferring argc from argv.
Reviewers: eugenis, vitalybuka
Reviewed By: eugenis
Subscribers: dberris, kristof.beyls, danielkiss, #sanitizers, delcypher
Tags: #sanitizers
Differential Revision: https://reviews.llvm.org/D77400
Diffstat (limited to 'llvm/lib/CodeGen/LiveDebugVariables.cpp')
0 files changed, 0 insertions, 0 deletions
