[all-commits] [llvm/llvm-project] 820e84: [RISCV] Assert initial load/store SEW is the EEW

Philip Reames via All-commits all-commits at lists.llvm.org
Mon Jun 20 07:45:39 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 820e84e05017d9c58074422e49dc59e27e1cc346
      https://github.com/llvm/llvm-project/commit/820e84e05017d9c58074422e49dc59e27e1cc346
  Author: Philip Reames <preames at rivosinc.com>
  Date:   2022-06-20 (Mon, 20 Jun 2022)

  Changed paths:
    M llvm/lib/Target/RISCV/RISCVInsertVSETVLI.cpp

  Log Message:
  -----------
  [RISCV] Assert initial load/store SEW is the EEW

In D127983, I had flipped from using the computed EEW to using the SEW value pulled from the VSETVLI when checking compatibility. This wasn't intentional, though thankfully it appears to be a non-functional difference. The new code does make a unchecked assumption that the initial SEW operand on the load/store is the EEW. This patch clarifies the assumption, and adds an assert to make sure this remains true.

Differential Revision: https://reviews.llvm.org/D128085




More information about the All-commits mailing list