6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2025-09-24 23:28:41 +00:00
Code Issues Projects Releases Wiki Activity

Slightly change the way stackmap and patchpoint intrinsics are lowered.

Browse Source 
MorphNodeTo is not safe to call during DAG building. It eagerly
deletes dependent DAG nodes which invalidates the NodeMap. We could
expose a safe interface for morphing nodes, but I don't think it's
worth it. Just create a new MachineNode and replaceAllUsesWith.

My understaning of the SD design has been that we want to support
early target opcode selection. That isn't very well supported, but
generally works. It seems reasonable to rely on this feature even if
it isn't widely used.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@194102 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Andrew Trick
2013-11-05 22:44:04 +00:00
parent 14cfb10174
commit dc8224def3
2 changed files with 47 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

36
lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
View File

@@ -6806,11 +6806,19 @@ void SelectionDAGBuilder::visitStackmap(const CallInst &CI) {
if (hasGlue)
Ops.push_back(*(Call->op_end()-1));
// Replace the target specific call node with STACKMAP in-place. This way we
// don't have to call ReplaceAllUsesWith and STACKMAP will take the call's
// place in the chain.
SDVTList NodeTys = DAG.getVTList(MVT::Other, MVT::Glue);
DAG.SelectNodeTo(Call, TargetOpcode::STACKMAP, NodeTys, &Ops[0], Ops.size());
// Replace the target specific call node with a STACKMAP node.
MachineSDNode *MN = DAG.getMachineNode(TargetOpcode::STACKMAP, getCurSDLoc(),
NodeTys, Ops);
// StackMap generates no value, so nothing goes in the NodeMap.
// Fixup the consumers of the intrinsic. The chain and glue may be used in the
// call sequence.
DAG.ReplaceAllUsesWith(Call, MN);
DAG.DeleteNode(Call);
}
/// \brief Lower llvm.experimental.patchpoint directly to its target opcode.
@@ -6898,12 +6906,22 @@ void SelectionDAGBuilder::visitPatchpoint(const CallInst &CI) {
if (hasGlue)
Ops.push_back(*(Call->op_end()-1));
// Replace the target specific call node with PATCHPOINT in-place. This
// way we don't have to call ReplaceAllUsesWith and PATCHPOINT will
// take the call's place in the chain.
SDVTList NodeTys = DAG.getVTList(MVT::Other, MVT::Glue);
DAG.SelectNodeTo(Call, TargetOpcode::PATCHPOINT, NodeTys, &Ops[0],
Ops.size());
// Replace the target specific call node with a STACKMAP node.
MachineSDNode *MN = DAG.getMachineNode(TargetOpcode::PATCHPOINT,
getCurSDLoc(), NodeTys, Ops);
// PatchPoint generates no value, so nothing goes in the NodeMap.
//
// FIXME: with anyregcc calling convention it will need to be in the NodeMap
// and replace values.
// Fixup the consumers of the intrinsic. The chain and glue may be used in the
// call sequence.
DAG.ReplaceAllUsesWith(Call, MN);
DAG.DeleteNode(Call);
}
/// TargetLowering::LowerCallTo - This is the default LowerCallTo