Re: Re: [PATCH V3] RISC-V: Support Dynamic LMUL Cost model

2023-09-11 Thread juzhe.zh...@rivai.ai
o.cheng; jeffreyalaw Subject: Re: [PATCH V3] RISC-V: Support Dynamic LMUL Cost model Hi Juzhe, glad that we can use the dominator info directly. Could we move the calculation of the info to the beginning (if it's not available)? That makes it clearer that it's a prerequisite. Funct

Re: Re: [PATCH V3] RISC-V: Support Dynamic LMUL Cost model

2023-09-11 Thread juzhe.zh...@rivai.ai
[PATCH V3] RISC-V: Support Dynamic LMUL Cost model Hi Juzhe, glad that we can use the dominator info directly. Could we move the calculation of the info to the beginning (if it's not available)? That makes it clearer that it's a prerequisite. Function comments look good now. Some gen

Re: [PATCH V3] RISC-V: Support Dynamic LMUL Cost model

2023-09-11 Thread Robin Dapp via Gcc-patches
Hi Juzhe, glad that we can use the dominator info directly. Could we move the calculation of the info to the beginning (if it's not available)? That makes it clearer that it's a prerequisite. Function comments look good now. Some general remarks kind of similar to v1: - I would prefer a hash

[PATCH V3] RISC-V: Support Dynamic LMUL Cost model

2023-09-11 Thread Juzhe-Zhong
This patch support dynamic LMUL cost modeling with --param=riscv-autovec-lmul=dynamic. Consider this following case: void foo (int32_t *__restrict a, int32_t *__restrict b,int32_t *__restrict c, int32_t *__restrict a2, int32_t *__restrict b2, int32_t *__restrict c2, int32_t *__res