diff options
author | Luke Lau <luke@igalia.com> | 2024-07-12 09:40:20 +0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-07-12 09:40:20 +0800 |
commit | ff8a03a7acca35eeba15bf4f1af8afa009cd04f3 (patch) | |
tree | 50acd2fdd9d4b47711e5c4114ad01754b55cf56b /clang/lib/CodeGen/CodeGenModule.h | |
parent | 79bd6287a9075efd6ef18755d27c540e744a892c (diff) | |
download | llvm-ff8a03a7acca35eeba15bf4f1af8afa009cd04f3.zip llvm-ff8a03a7acca35eeba15bf4f1af8afa009cd04f3.tar.gz llvm-ff8a03a7acca35eeba15bf4f1af8afa009cd04f3.tar.bz2 |
[RISCV] Fix coalesced vsetvli's AVL LiveInterval not always being shrunk (#98286)
Most of the time when we coalesce and delete a vsetvli, we shrink the
LiveInterval of its AVL register now that there is one less use. However
there's one edge case we were missing where if we have two vsetvlis with
no users of vl or vtype in between, we coalesced a vsetvli without
shrinking it's AVL.
This fixes it by shrinking the LiveInterval whenever we delete a
vsetvli, and also makes the LiveIntervals consistent in-situ by not
removing the use before shrinking.
This fixes a -verify-machineinstrs assertion in an MIR test case I found
while investigating
https://github.com/llvm/llvm-project/pull/97264#issuecomment-2218036877.
I couldn't recreate this at the LLVM IR level, seemingly because
RISCVInsertVSETVLI will just avoid inserting extra vsetvlis that don't
need coalesced.
Diffstat (limited to 'clang/lib/CodeGen/CodeGenModule.h')
0 files changed, 0 insertions, 0 deletions