Committed after the commited of the vsetvl pass refactor patch, thanks
Robin.
On 2023/10/19 16:43, Robin Dapp wrote:
Hi Juzhe,
as discussed off-list this approach generally makes sense to me so
the patch LGTM once the vsetvl rework is upstream and settled.
Independently, we still need to unde
instructions are considered as very cheap (COST = 1).
juzhe.zh...@rivai.ai
From: Robin Dapp
Date: 2023-10-19 16:43
To: Juzhe-Zhong; gcc-patches
CC: rdapp.gcc; kito.cheng; kito.cheng; jeffreyalaw
Subject: Re: [PATCH V2] RISC-V: Fix failed hoist in LICM of vmv.v.x instruction
Hi Juzhe,
as discussed off
Hi Juzhe,
as discussed off-list this approach generally makes sense to me so
the patch LGTM once the vsetvl rework is upstream and settled.
Independently, we still need to understand why the more complex
broadcast pattern is not hoisted out of the loop.
Regards
Robin
More details of VSETVL bug:
Loop:
10ddc: 9ed030d7vmv1r.v v1,v13
10de0: b21040d7vncvt.x.x.w v1,v1
10de4: 5e0785d7vmv.v.v v11,v15
10de8: b700a5d7vmacc.vvv11,v1,v16
10dec: a6e8a0d7
