Re: Re: [PATCH] RISC-V: Disable AVL propagation of vrgather instruction

2023-11-23 Thread juzhe.zh...@rivai.ai
Date: 2023-11-23 19:58 To: juzhe.zh...@rivai.ai; gcc-patches CC: rdapp.gcc; kito.cheng; Kito.cheng; jeffreyalaw Subject: Re: [PATCH] RISC-V: Disable AVL propagation of vrgather instruction > Oh. You mean this patch also fixes FLTO failed case ? Yes, it's the same issue. There we have a fixed

Re: [PATCH] RISC-V: Disable AVL propagation of vrgather instruction

2023-11-23 Thread Robin Dapp
> Oh. You mean this patch also fixes FLTO failed case ? Yes, it's the same issue. There we have a fixed vl (known via LTO) that is being propagated "into" gathers and we end up missing gather elements. Regards Robin

Re: Re: [PATCH] RISC-V: Disable AVL propagation of vrgather instruction

2023-11-23 Thread juzhe.zh...@rivai.ai
Oh. You mean this patch also fixes FLTO failed case ? juzhe.zh...@rivai.ai From: Robin Dapp Date: 2023-11-23 19:55 To: Juzhe-Zhong; gcc-patches CC: rdapp.gcc; kito.cheng; kito.cheng; jeffreyalaw Subject: Re: [PATCH] RISC-V: Disable AVL propagation of vrgather instruction I was just about to

Re: [PATCH] RISC-V: Disable AVL propagation of vrgather instruction

2023-11-23 Thread Robin Dapp
I was just about to post a similar-ish patch that fixes pr65518.c but you were faster ;) Therefore LGTM. You can add PR/target 112670. Regards Robin