Skip to content

Commit

Permalink
[TargetLowering] Remove comment above definition of TargetLowering::E…
Browse files Browse the repository at this point in the history
…mitInstrWithCustomInserter. NFC

This duplicates the documentation from the declaration, but has
gotten out of sync. There was a mention of a DenseMap that has
been gone for 14 years. Since this declaration is a default
implementation of a virtual function, its unlikely to be looked at.
Just remove it.
  • Loading branch information
topperc committed Jun 29, 2024
1 parent c00ada0 commit 9ea9e71
Showing 1 changed file with 0 additions and 9 deletions.
9 changes: 0 additions & 9 deletions llvm/lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -311,15 +311,6 @@ namespace llvm {

} // end namespace llvm

// EmitInstrWithCustomInserter - This method should be implemented by targets
// that mark instructions with the 'usesCustomInserter' flag. These
// instructions are special in various ways, which require special support to
// insert. The specified MachineInstr is created but not inserted into any
// basic blocks, and this method is called to expand it into a sequence of
// instructions, potentially also creating new basic blocks and control flow.
// When new basic blocks are inserted and the edges from MBB to its successors
// are modified, the method should insert pairs of <OldSucc, NewSucc> into the
// DenseMap.
MachineBasicBlock *
TargetLowering::EmitInstrWithCustomInserter(MachineInstr &MI,
MachineBasicBlock *MBB) const {
Expand Down

0 comments on commit 9ea9e71

Please sign in to comment.