[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