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

[CodeGen] Hide SparseSet behind a typedef (NFC) (#167898)

2025-11-13T17:14:23+00:00

So that changing the type of the container (planned in a future patch)
is less intrusive.

[MachineCombiner][Targets] Use Register in TII genAlternativeCodeSequence interface. NFC (#131272)

2025-03-14T06:27:56+00:00

[NFC] Remove some unnecessary semicolons

2024-12-16T08:48:57+00:00

All inside LLVM_DEBUG, some of which have been cleaned up by adding block
scopes to allow them to format more nicely.

Check hasOptSize() in shouldOptimizeForSize() (#112626)

2024-10-28T16:45:03+00:00

[CodeGen][NewPM] Port machine trace metrics analysis to new pass manager. (#108507)

2024-10-16T07:49:55+00:00

[CodeGen][NewPM] Port `machine-loops` to new pass manager (#97793)

2024-07-09T01:11:18+00:00

- Add `MachineLoopAnalysis`.
- Add `MachineLoopPrinterPass`.
- Convert to `MachineLoopInfoWrapperPass` in legacy pass manager.

[CodeGen][NewPM] Split `MachineDominatorTree` into a concrete analysis result (#94571)

2024-06-11T13:27:14+00:00

Prepare for new pass manager version of `MachineDominatorTreeAnalysis`.
We may need a machine dominator tree version of `DomTreeUpdater` to
handle `SplitCriticalEdge` in some CodeGen passes.

[CodeGen] Make the parameter TRI required in some functions. (#85968)

2024-04-24T13:24:14+00:00

Fixes #82659

There are some functions, such as `findRegisterDefOperandIdx` and  `findRegisterDefOperand`, that have too many default parameters. As a result, we have encountered some issues due to the lack of TRI  parameters, as shown in issue #82411.

Following @RKSimon 's suggestion, this patch refactors 9 functions, including `{reads, kills, defines, modifies}Register`,  `registerDefIsDead`, and `findRegister{UseOperandIdx, UseOperand, DefOperandIdx, DefOperand}`, adjusting the order of the TRI parameter and making it required. In addition, all the places that call these functions have also been updated correctly to ensure no additional impact.

After this, the caller of these functions should explicitly know whether to pass the `TargetRegisterInfo` or just a `nullptr`.

[MachineCombiner][NFC] Split target-dependent patterns

2024-04-11T04:20:27+00:00

We split target-dependent MachineCombiner patterns into their target
folder.

This makes MachineCombiner much more target-independent.

Reviewers:
davemgreen, asavonic, rotateright, RKSimon, lukel97, LuoYuanke, topperc, mshockwave, asi-sc

Reviewed By: topperc, mshockwave

Pull Request: https://github.com/llvm/llvm-project/pull/87991

[MachineCombiner] Don't ignore PHI depths (#82025)

2024-03-14T15:47:29+00:00

The depths of the Root and the NewRoot are to be compared in
MachineCombiner::improvesCriticalPathLen(), and while the call to
BlockTrace.getInstrCycles(*Root) includes the Depth of a PHI, for some
reason PHI nodes have been ignored in getOperandDef(). 

This patch removes the special handling of PHIs in getOperandDef() so that
Root and NewRoot get a fair comparison. This does not affect loop headers
as MachineTraceMetrics handles that case by ignoring incoming PHI edges.