Re: LTO: Capture 'lto_file_decl_data *file_data' in 'class lto_input_block' (was: [PATCH v3] Streamer: Fix out of range memory access of machine mode)

2023-06-30 Thread Richard Biener via Gcc-patches
On Fri, 30 Jun 2023, Thomas Schwinge wrote: > Hi! > > On 2023-06-29T22:14:59+0200, I wrote: > > [the new] 'file_data->mode_bits' needs to be considered [somewhere] > > > > Easiest is in 'gcc/lto-streamer.h:class lto_input_block' to capture > > 'lto_file_decl_data *file_data' instead of just > > '

LTO: Capture 'lto_file_decl_data *file_data' in 'class lto_input_block' (was: [PATCH v3] Streamer: Fix out of range memory access of machine mode)

2023-06-30 Thread Thomas Schwinge
Hi! On 2023-06-29T22:14:59+0200, I wrote: > [the new] 'file_data->mode_bits' needs to be considered [somewhere] > > Easiest is in 'gcc/lto-streamer.h:class lto_input_block' to capture > 'lto_file_decl_data *file_data' instead of just > 'unsigned char *mode_table', and adjust all users. I've split

RE: Re: [PATCH v3] Streamer: Fix out of range memory access of machine mode

2023-06-29 Thread Li, Pan2 via Gcc-patches
, Yanzhang ; kito.cheng ; Tobias Burnus Subject: Re: Re: [PATCH v3] Streamer: Fix out of range memory access of machine mode Thanks a lot! Really appreciate your help ! That's really helpful for RVV (RISC-V vector). Could you merge your patch after you tested? T

Re: Re: [PATCH v3] Streamer: Fix out of range memory access of machine mode

2023-06-29 Thread juzhe.zh...@rivai.ai
he.zh...@rivai.ai; rdapp@gmail.com; jeffreya...@gmail.com; yanzhang.w...@intel.com; kito.ch...@gmail.com; Tobias Burnus Subject: Re: [PATCH v3] Streamer: Fix out of range memory access of machine mode Hi! On 2023-06-29T11:29:57+0200, I wrote: > On 2023-06-21T15:58:24+0800, Pan Li via Gcc

Re: [PATCH v3] Streamer: Fix out of range memory access of machine mode

2023-06-29 Thread Thomas Schwinge
Hi! On 2023-06-29T11:29:57+0200, I wrote: > On 2023-06-21T15:58:24+0800, Pan Li via Gcc-patches > wrote: >> We extend the machine mode from 8 to 16 bits already. But there still >> one placing missing from the streamer. It has one hard coded array >> for the machine code like size 256. >> >> In

Re: Re: [PATCH v3] Streamer: Fix out of range memory access of machine mode

2023-06-29 Thread juzhe.zh...@rivai.ai
gcc-patches@gcc.gnu.org; Robin Dapp; jeffreya...@gmail.com; yanzhang.w...@intel.com; kito.ch...@gmail.com; rguent...@suse.de; ja...@redhat.com; Tobias Burnus Subject: Re: Re: [PATCH v3] Streamer: Fix out of range memory access of machine mode Hi! On 2023-06-29T17:33:14+0800, "juzhe.zh...@r

Re: Re: [PATCH v3] Streamer: Fix out of range memory access of machine mode

2023-06-29 Thread Thomas Schwinge
8) > FAIL: gcc.target/riscv/rvv/autovec/partial/slp_run-3.c (test for excess > errors) That looks like a different issue, though? Grüße Thomas > From: Thomas Schwinge > Date: 2023-06-29 17:29 > To: Pan Li > CC: gcc-patches@gcc.gnu.org; juzhe.zh...@rivai.ai; rdapp....@gmail.com;

Re: Re: [PATCH v3] Streamer: Fix out of range memory access of machine mode

2023-06-29 Thread juzhe.zh...@rivai.ai
Subject: Re: [PATCH v3] Streamer: Fix out of range memory access of machine mode Hi! On 2023-06-21T15:58:24+0800, Pan Li via Gcc-patches wrote: > We extend the machine mode from 8 to 16 bits already. But there still > one placing missing from the streamer. It has one hard coded array >

Re: [PATCH v3] Streamer: Fix out of range memory access of machine mode

2023-06-29 Thread Thomas Schwinge
Hi! On 2023-06-21T15:58:24+0800, Pan Li via Gcc-patches wrote: > We extend the machine mode from 8 to 16 bits already. But there still > one placing missing from the streamer. It has one hard coded array > for the machine code like size 256. > > In the lto pass, we memset the array by MAX_MACHIN

RE: [PATCH v3] Streamer: Fix out of range memory access of machine mode

2023-06-22 Thread Li, Pan2 via Gcc-patches
g Cc: juzhe.zh...@rivai.ai; rdapp@gmail.com; jeffreya...@gmail.com; Li, Pan2 ; Wang, Yanzhang ; kito.ch...@gmail.com; rguent...@suse.de; ja...@redhat.com Subject: [PATCH v3] Streamer: Fix out of range memory access of machine mode From: Pan Li We extend the machine mode from 8 to 16 bits already.

[PATCH v3] Streamer: Fix out of range memory access of machine mode

2023-06-21 Thread Pan Li via Gcc-patches
From: Pan Li We extend the machine mode from 8 to 16 bits already. But there still one placing missing from the streamer. It has one hard coded array for the machine code like size 256. In the lto pass, we memset the array by MAX_MACHINE_MODE count but the value of the MAX_MACHINE_MODE will grow