llvm-project.git/llvm/lib/CodeGen/StackSlotColoring.cpp, branch main

[StackSlotColoring] Fix issue where colors for a StackID are dropped (#138140)

2025-05-02T09:23:39+00:00

In InitializeSlots, if an interval with a non-zero StackID (A) is
encountered we set All/UsedColors to a size of A + 1. If after this we
process another interval with a non-zero StackID (B), where B < A, then
we resize All/UsedColors to size < A + 1, and lose the BitVector
associated with A.

AFAIK this is a latent bug upstream, but when adding a new TargetStackID
locally I hit a `NextColors[StackID] != -1 && "No more spill slots?"`
assertion due to this issue.

[TTI] Use TypeSize in isLoadFromStackSlot and isStoreToStackSlot [nfc] (#132244)

2025-03-20T17:17:36+00:00

Motivation is supporting scalable spills and reloads, e.g. in
https://github.com/llvm/llvm-project/pull/120524.

Looking at this API, I'm suspicious that the access size should just be
coming from the memory operand on the load or store, but we don't appear
to be consistently setting that up. That's a larger change so I may or
may not bother pursuing that.

[CodeGen][NewPM] Port StackSlotColoring to NPM. (#125876)

2025-02-05T17:48:16+00:00

[CodeGen] Add Register::stackSlotIndex(). Replace uses of Register::stackSlot2Index. NFC (#125028)

2025-01-30T07:02:07+00:00

[CodeGen][NewPM] Port LiveStacks analysis to NPM (#118778)

2024-12-06T09:46:07+00:00

[CodeGen][NewPM] Port LiveDebugVariables to NPM (#115468)

2024-12-04T09:01:34+00:00

The existing analysis was already a pimpl wrapper.

I have extracted legacy pass logic to a LDVImpl wrapper named
`LiveDebugVariables` which is the analysis::Result now. This controls
whether to activate the LDV (depending on `-live-debug-variables` and
DIsubprogram) itself.

The legacy and new analysis only construct the LiveDebugVariables.

VirtRegRewriter will test this.

[CodeGen] Remove unused includes (NFC) (#115996)

2024-11-13T07:15:06+00:00

Identified with misc-include-cleaner.

[StackSlotColoring] Use Register for isLoadFromStackSlot/isStoreToStackSlot result. NFC

2024-09-15T19:05:28+00:00

[CodeGen] Use range-based for loops (NFC) (#98706)

2024-07-13T20:29:47+00:00

[CodeGen][NewPM] Port `machine-block-freq` to new pass manager (#98317)

2024-07-12T07:45:01+00:00

- Add `MachineBlockFrequencyAnalysis`.
- Add `MachineBlockFrequencyPrinterPass`.
- Use `MachineBlockFrequencyInfoWrapperPass` in legacy pass manager.
- `LazyMachineBlockFrequencyInfo::print` is empty, drop it due to new
pass manager migration.