Updates to support

* Changes in PHI node structure
* Fix to Predecessor iterator

llvm-svn: 23
This commit is contained in:
Chris Lattner 2001-06-11 15:04:06 +00:00
parent 7399a10ad5
commit a653daaac2
3 changed files with 17 additions and 20 deletions

View File

@ -106,7 +106,7 @@ inline df_const_iterator df_end (const BasicBlock *BB);
template <class _Ptr, class _USE_iterator> // Predecessor Iterator template <class _Ptr, class _USE_iterator> // Predecessor Iterator
class PredIterator { class PredIterator {
const _Ptr ThisBB; const _Ptr BB;
_USE_iterator It; _USE_iterator It;
public: public:
typedef PredIterator<_Ptr,_USE_iterator> _Self; typedef PredIterator<_Ptr,_USE_iterator> _Self;
@ -116,15 +116,16 @@ public:
inline void advancePastConstPool() { inline void advancePastConstPool() {
// Loop to ignore constant pool references // Loop to ignore constant pool references
while (It != ThisBB->use_end() && while (It != BB->use_end() &&
((*It)->getValueType() != Value::InstructionVal)) (((*It)->getValueType() != Value::InstructionVal) ||
!(((Instruction*)(*It))->isTerminator())))
++It; ++It;
} }
inline PredIterator(_Ptr BB) : ThisBB(BB), It(BB->use_begin()) { inline PredIterator(_Ptr bb) : BB(bb), It(bb->use_begin()) {
advancePastConstPool(); advancePastConstPool();
} }
inline PredIterator(_Ptr BB, bool) : ThisBB(BB), It(BB->use_end()) {} inline PredIterator(_Ptr bb, bool) : BB(bb), It(bb->use_end()) {}
inline bool operator==(const _Self& x) const { return It == x.It; } inline bool operator==(const _Self& x) const { return It == x.It; }
inline bool operator!=(const _Self& x) const { return !operator==(x); } inline bool operator!=(const _Self& x) const { return !operator==(x); }

View File

@ -99,7 +99,7 @@ public:
inline UseTy<ValueSubclass>(const UseTy<ValueSubclass> &user) { inline UseTy<ValueSubclass>(const UseTy<ValueSubclass> &user) {
Val = 0; Val = 0;
U = user.U; U = user.U;
operator=(user); operator=(user.Val);
} }
inline ValueSubclass *operator=(ValueSubclass *V) { inline ValueSubclass *operator=(ValueSubclass *V) {
if (Val) Val->killUse(U); if (Val) Val->killUse(U);

View File

@ -20,16 +20,10 @@
// node, that can not exist in nature, but can be synthesized in a computer // node, that can not exist in nature, but can be synthesized in a computer
// scientist's overactive imagination. // scientist's overactive imagination.
// //
// TODO: FIXME: This representation is not good enough. Consider the following
// code:
// BB0: %x = int %0
// BB1: %y = int %1
// BB2: %z = phi int %0, %1 - Can't tell where constants come from!
//
// TOFIX: Store pair<Use,BasicBlockUse> instead of just <Use>
//
class PHINode : public Instruction { class PHINode : public Instruction {
vector<Use> IncomingValues; typedef pair<Use,BasicBlockUse> PairTy;
vector<PairTy> IncomingValues;
PHINode(const PHINode &PN); PHINode(const PHINode &PN);
public: public:
PHINode(const Type *Ty, const string &Name = ""); PHINode(const Type *Ty, const string &Name = "");
@ -40,22 +34,24 @@ public:
// Implement all of the functionality required by User... // Implement all of the functionality required by User...
// //
virtual void dropAllReferences(); virtual void dropAllReferences();
virtual const Value *getOperand(unsigned i) const { virtual const Value *getOperand(unsigned i) const {
return (i < IncomingValues.size()) ? IncomingValues[i] : 0; if (i >= IncomingValues.size()*2) return 0;
if (i & 1) return IncomingValues[i/2].second;
else return IncomingValues[i/2].first;
} }
inline Value *getOperand(unsigned i) { inline Value *getOperand(unsigned i) {
return (Value*)((const PHINode*)this)->getOperand(i); return (Value*)((const PHINode*)this)->getOperand(i);
} }
virtual unsigned getNumOperands() const { return IncomingValues.size(); } virtual unsigned getNumOperands() const { return IncomingValues.size()*2; }
virtual bool setOperand(unsigned i, Value *Val); virtual bool setOperand(unsigned i, Value *Val);
virtual string getOpcode() const { return "phi"; } virtual string getOpcode() const { return "phi"; }
// addIncoming - Add an incoming value to the end of the PHI list // addIncoming - Add an incoming value to the end of the PHI list
void addIncoming(Value *D); void addIncoming(Value *D, BasicBlock *BB);
// removeIncomingValue - Remove an incoming value. This is useful if a // removeIncomingValue - Remove an incoming value. This is useful if a
// predecessor basic block is deleted. The value removed is returned. // predecessor basic block is deleted. The value removed is returned.
Value *removeIncomingValue(unsigned idx); Value *removeIncomingValue(const BasicBlock *BB);
}; };