RPCS3/llvm-mirror
1
0
Fork 0
mirror of https://github.com/RPCS3/llvm-mirror.git synced 2025-02-01 13:11:39 +01:00
Code Issues Projects Releases Wiki Activity

Minor cosmestic changes (NFC)

Browse Source 
- Reflow comments
 - Rename function

llvm-svn: 264319
This commit is contained in:
Sanjoy Das 2016-03-24 18:57:31 +00:00
parent c6ea540077
commit 4d8366ac18
2 changed files with 12 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
include/llvm/CodeGen/FunctionLoweringInfo.h
View File

@ -80,12 +80,11 @@ public:
/// Track virtual registers created for exception pointers.
DenseMap<const Value *, unsigned> CatchPadExceptionPointers;
// Keep track of frame indices allocated for statepoints as they could be used
// across basic block boundaries.
// Key of the map is statepoint instruction, value is a map from spilled
// llvm Value to the optional stack stack slot index.
// If optional is unspecified it means that we have visited this value
// but didn't spill it.
/// Keep track of frame indices allocated for statepoints as they could be
/// used across basic block boundaries. Key of the map is statepoint
/// instruction, value is a map from spilled llvm Value to the optional stack
/// stack slot index. If optional is unspecified it means that we have
/// visited this value but didn't spill it.
typedef DenseMap<const Value*, Optional<int>> StatepointSpilledValueMapTy;
DenseMap<const Instruction*, StatepointSpilledValueMapTy>
StatepointRelocatedValues;

14
lib/CodeGen/SelectionDAG/StatepointLowering.cpp
View File

@ -246,10 +246,10 @@ static void reservePreviousStackSlotForValue(const Value *IncomingValue,
/// StackMap section. It has no effect on the number of spill slots required
/// or the actual lowering.
static void
removeDuplicatesGCPtrs(SmallVectorImpl<const Value *> &Bases,
SmallVectorImpl<const Value *> &Ptrs,
SmallVectorImpl<const GCRelocateInst *> &Relocs,
SelectionDAGBuilder &Builder) {
removeDuplicateGCPtrs(SmallVectorImpl<const Value *> &Bases,
SmallVectorImpl<const Value *> &Ptrs,
SmallVectorImpl<const GCRelocateInst *> &Relocs,
SelectionDAGBuilder &Builder) {
// This is horribly inefficient, but I don't care right now
SmallSet<SDValue, 32> Seen;
@ -540,8 +540,8 @@ SDValue SelectionDAGBuilder::LowerAsSTATEPOINT(
StatepointLowering.startNewStatepoint(*this);
#ifndef NDEBUG
// We schedule gc relocates before removeDuplicatesGCPtrs since we _will_
// encounter the duplicate gc relocates we elide in removeDuplicatesGCPtrs.
// We schedule gc relocates before removeDuplicateGCPtrs since we _will_
// encounter the duplicate gc relocates we elide in removeDuplicateGCPtrs.
for (auto *Reloc : SI.GCRelocates)
if (Reloc->getParent() == SI.StatepointInstr->getParent())
StatepointLowering.scheduleRelocCall(*Reloc);
@ -551,7 +551,7 @@ SDValue SelectionDAGBuilder::LowerAsSTATEPOINT(
// input. Also has the effect of removing duplicates in the original
// llvm::Value input list as well. This is a useful optimization for
// reducing the size of the StackMap section. It has no other impact.
removeDuplicatesGCPtrs(SI.Bases, SI.Ptrs, SI.GCRelocates, *this);
removeDuplicateGCPtrs(SI.Bases, SI.Ptrs, SI.GCRelocates, *this);
assert(SI.Bases.size() == SI.Ptrs.size() &&
SI.Ptrs.size() == SI.GCRelocates.size());