Added more comments and committed, thanks! On Mon, Jan 23, 2023 at 3:36 AM Jeff Law wrote: > > > On 1/17/23 20:03, juzhe.zhong@rivai.ai wrote: > > From: Ju-Zhe Zhong > > > > gcc/ChangeLog: > > > > * config/riscv/riscv-passes.def (INSERT_PASS_BEFORE): Reorder > VSETVL PASS. > It'd be useful to know the motivation here, I could easily see someone > in the future finding a case where the location of the vsetvl gets > influenced by some dead code that would have been removed by the DCE > pass. Then they're going to want to know the motivation behind the > current pass placement. > > So, OK after adding a comment describing why the pass is placed where it > is. > > Jeff >