forked from OSchip/llvm-project
Remove the 'Parent' pointer from the MDNodeOperand class.
An MDNode has a list of MDNodeOperands allocated directly after it as part of its allocation. Therefore, the Parent of the MDNodeOperands can be found by walking back through the operands to the beginning of that list. Mark the first operand's value pointer as being the 'first' operand so that we know where the beginning of said list is. This saves a *lot* of space during LTO with -O0 -g flags. llvm-svn: 154280
This commit is contained in:
parent
9b2503a006
commit
5c0068f807
|
@ -50,14 +50,26 @@ MDString *MDString::get(LLVMContext &Context, StringRef Str) {
|
|||
// Use CallbackVH to hold MDNode operands.
|
||||
namespace llvm {
|
||||
class MDNodeOperand : public CallbackVH {
|
||||
MDNode *Parent;
|
||||
MDNode *getParent() {
|
||||
MDNodeOperand *Cur = this;
|
||||
|
||||
while (Cur->getValPtrInt() != 1)
|
||||
--Cur;
|
||||
|
||||
assert(Cur->getValPtrInt() == 1 &&
|
||||
"Couldn't find the beginning of the operand list!");
|
||||
return reinterpret_cast<MDNode*>(Cur) - 1;
|
||||
}
|
||||
|
||||
public:
|
||||
MDNodeOperand(Value *V, MDNode *P) : CallbackVH(V), Parent(P) {}
|
||||
MDNodeOperand(Value *V) : CallbackVH(V) {}
|
||||
~MDNodeOperand() {}
|
||||
|
||||
void set(Value *V) {
|
||||
setValPtr(V);
|
||||
}
|
||||
void set(Value *V) { this->setValPtr(V); }
|
||||
|
||||
/// setAsFirstOperand - Accessor method to mark the operand as the first in
|
||||
/// the list.
|
||||
void setAsFirstOperand(unsigned V) { this->setValPtrInt(V); }
|
||||
|
||||
virtual void deleted();
|
||||
virtual void allUsesReplacedWith(Value *NV);
|
||||
|
@ -66,15 +78,13 @@ public:
|
|||
|
||||
|
||||
void MDNodeOperand::deleted() {
|
||||
Parent->replaceOperand(this, 0);
|
||||
getParent()->replaceOperand(this, 0);
|
||||
}
|
||||
|
||||
void MDNodeOperand::allUsesReplacedWith(Value *NV) {
|
||||
Parent->replaceOperand(this, NV);
|
||||
getParent()->replaceOperand(this, NV);
|
||||
}
|
||||
|
||||
|
||||
|
||||
//===----------------------------------------------------------------------===//
|
||||
// MDNode implementation.
|
||||
//
|
||||
|
@ -102,8 +112,13 @@ MDNode::MDNode(LLVMContext &C, ArrayRef<Value*> Vals, bool isFunctionLocal)
|
|||
// Initialize the operand list, which is co-allocated on the end of the node.
|
||||
unsigned i = 0;
|
||||
for (MDNodeOperand *Op = getOperandPtr(this, 0), *E = Op+NumOperands;
|
||||
Op != E; ++Op, ++i)
|
||||
new (Op) MDNodeOperand(Vals[i], this);
|
||||
Op != E; ++Op, ++i) {
|
||||
new (Op) MDNodeOperand(Vals[i]);
|
||||
|
||||
// Mark the first MDNodeOperand as being the first in the list of operands.
|
||||
if (i == 0)
|
||||
Op->setAsFirstOperand(1);
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
|
|
Loading…
Reference in New Issue