[all-commits] [llvm/llvm-project] 115331: [LocalStackSlotAllocation] Minor simplifications. NFC

Craig Topper via All-commits all-commits at lists.llvm.org
Mon Jan 9 09:50:27 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 1153313c334aa0ebe40b26593bd689a779ecaa23
      https://github.com/llvm/llvm-project/commit/1153313c334aa0ebe40b26593bd689a779ecaa23
  Author: Craig Topper <craig.topper at sifive.com>
  Date:   2023-01-09 (Mon, 09 Jan 2023)

  Changed paths:
    M llvm/lib/CodeGen/LocalStackSlotAllocation.cpp

  Log Message:
  -----------
  [LocalStackSlotAllocation] Minor simplifications. NFC

Instead of maintaining a separate valid flag for BaseReg, Use
BaseReg.isValid(). I think this is left over from an older
implementation that maintained a vector of base registers.

The other change is not do a speculative assignment to BaseOffset
that needs to be reverted. Only commit it after we do the check.

Reviewed By: arsenm

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




More information about the All-commits mailing list