[PATCH] D125232: [riscv] Use classic dataflow for VSETVLI insertion

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 16 14:17:18 PDT 2022


reames added a comment.

In D125232#3517207 <https://reviews.llvm.org/D125232#3517207>, @craig.topper wrote:

> Is enabling strict asserts going to be a separate patch?

D125271 <https://reviews.llvm.org/D125271>



================
Comment at: llvm/lib/Target/RISCV/RISCVInsertVSETVLI.cpp:973
 void RISCVInsertVSETVLI::computeIncomingVLVTYPE(const MachineBasicBlock &MBB) {
+
   BlockData &BBInfo = BlockInfo[MBB.getNumber()];
----------------
craig.topper wrote:
> Any reason for the new blank line here?
Nope, thought I'd killed that in the last update. Will remove.


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

https://reviews.llvm.org/D125232



More information about the llvm-commits mailing list