[PATCH] D106857: [RISCV] Teach VSETVLI insertion to merge the unused VSETVLI with the one need to be insert after it.

Jianjian Guan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 29 00:04:13 PDT 2021


jacquesguan added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVInsertVSETVLI.cpp:677
           insertVSETVLI(MBB, MI, NewInfo, BlockInfo[MBB.getNumber()].Pred);
           CurInfo = NewInfo;
         }
----------------
frasercrmck wrote:
> Is this path not setting `VSetMI` by inserting a VSETVLI?
No need to set it here, `CurInfo` is definitely not from an explicit VSET(I)VLI of this MBB.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D106857/new/

https://reviews.llvm.org/D106857



More information about the llvm-commits mailing list