Re: [PATCH] riscv: update riscv_asan_shadow_offset

[Kito Cheng via Gcc-patches]

Andreas Schwab via Gcc-patches  於 2023年5月30日 週二
17:37 寫道：

> Ok for 12 and 13 branch?
>

Yes, thanks!


> --
> 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."
>

Re: [PATCH] riscv: update riscv_asan_shadow_offset

[Andreas Schwab via Gcc-patches]

Ok for 12 and 13 branch?

-- 
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."

Re: [PATCH] riscv: update riscv_asan_shadow_offset

[Kito Cheng via Gcc-patches]

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
 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 (0xd) : 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."

[PATCH] riscv: update riscv_asan_shadow_offset

[Andreas Schwab via Gcc-patches]

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 (0xd) : 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."