This patch replaces fprintf with print_address function in LSAN tests. This is necessary because of different printing of pointers in fprintf and sanitizer's print function. After this patch is committed, we can enable make check-lsan option on aarch64 architecture.
Details
Details
Diff Detail
Diff Detail
Event Timeline
test/lsan/lit.common.cfg | ||
---|---|---|
34–36 | For consistency, this should also be just /../ and the header should be included in LSan tests as #include "sanitizer_common/print_address.h". |
Comment Actions
This was committed a few days ago. I guess that the ending dot in the "Differential Revision" link prevented this issue to be automatically updated.
commit f2247b9c1a68a294ba6b027edc71808a5202ce3a
Author: Strahinja Petrovic <strahinja.petrovic@rt-rk.com>
Date: Mon Nov 14 11:40:56 2016 +0000
[lsan] [aarch64] Fix printing of pointers in make check tests - update This patch replaces fprintf with print_address function in LSAN tests. This is necessary because of different printing of pointers in fprintf and sanitizer's print function. Differential Revision: https://reviews.llvm.org/D26084.
For consistency, this should also be just /../ and the header should be included in LSan tests as #include "sanitizer_common/print_address.h".