LGTM, I remember Luís updated[1] that, but apparently I forgot sync this to gcc,
and just to remind, I plan to change that to dynamic offset[2] to make that work on Sv39, Sv48 and Sv57, but we are still running testing and debugging to make sure LSAN works well... [1] https://reviews.llvm.org/D97646 [2] https://reviews.llvm.org/D139827 On Tue, May 30, 2023 at 4:43 PM Andreas Schwab via Gcc-patches <gcc-patches@gcc.gnu.org> wrote: > > This fixes all asan tests, apart from > c-c++-common/asan/pointer-compare-1.c which needs a workaround for PR > sanitizer/82501. > > PR target/110036 > * config/riscv/riscv.cc (riscv_asan_shadow_offset): Update to > match libsanitizer. > --- > gcc/config/riscv/riscv.cc | 7 +++---- > 1 file changed, 3 insertions(+), 4 deletions(-) > > diff --git a/gcc/config/riscv/riscv.cc b/gcc/config/riscv/riscv.cc > index 09fc9e5d95e..b358ca8b5d0 100644 > --- a/gcc/config/riscv/riscv.cc > +++ b/gcc/config/riscv/riscv.cc > @@ -7043,10 +7043,9 @@ riscv_asan_shadow_offset (void) > { > /* We only have libsanitizer support for RV64 at present. > > - This number must match kRiscv*_ShadowOffset* in the file > - libsanitizer/asan/asan_mapping.h which is currently 1<<29 for rv64, > - even though 1<<36 makes more sense. */ > - return TARGET_64BIT ? (HOST_WIDE_INT_1 << 29) : 0; > + This number must match ASAN_SHADOW_OFFSET_CONST in the file > + libsanitizer/asan/asan_mapping.h. */ > + return TARGET_64BIT ? HOST_WIDE_INT_UC (0xd55550000) : 0; > } > > /* Implement TARGET_MANGLE_TYPE. */ > -- > 2.40.1 > > > -- > Andreas Schwab, SUSE Labs, sch...@suse.de > GPG Key fingerprint = 0196 BAD8 1CE9 1970 F4BE 1748 E4D4 88E3 0EEA B9D7 > "And now for something completely different."
