[all-commits] [llvm/llvm-project] 558397: [RISCV] Simplify eliminateFrameIndex in advance of...

Philip Reames via All-commits all-commits at lists.llvm.org
Mon Nov 28 10:09:54 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 5583972fe14e532c17d3d1a0607051e5adcbc7bb
      https://github.com/llvm/llvm-project/commit/5583972fe14e532c17d3d1a0607051e5adcbc7bb
  Author: Philip Reames <preames at rivosinc.com>
  Date:   2022-11-28 (Mon, 28 Nov 2022)

  Changed paths:
    M llvm/lib/Target/RISCV/RISCVRegisterInfo.cpp
    M llvm/lib/Target/RISCV/RISCVRegisterInfo.h
    M llvm/test/CodeGen/RISCV/rvv/addi-scalable-offset.mir
    M llvm/test/CodeGen/RISCV/rvv/emergency-slot.mir

  Log Message:
  -----------
  [RISCV] Simplify eliminateFrameIndex in advance of reuse [nfc-ish]

The prior code intermixed several concerns - the actual materialization of the offset, the choice of destination register, and whether to prune the ADDI. This version factors the first part out, and then reasons only about the later two. My intention is to merge the adjustReg routine with the one from frame lowering, and then explore using the merged result to simplify frame setup and tear down.

This change is conceptually NFC, but since it results in slightly different vreg usage, the end result can change register allocation in minor ways.

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




More information about the All-commits mailing list