> +  /// Return the immediate dominator of A.
> +  BasicBlock *getIDom(BasicBlock *A) {
> +    ETNode *NodeA = getNode(A);
> +    const ETNode *idom = NodeA->getFather();
> +    return idom ? idom->getData<BasicBlock>() : 0;
> +  }

Random other question: Why does ETNode have a template accessor to  
get its data?  Is the Data pointer every anything other than a  
BasicBlock*?  If not, why not just type the pointer as BasicBlock*  
and rename getData() -> getBasicBlock() ?

-Chris
_______________________________________________
llvm-commits mailing list
llvm-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

Reply via email to