[llvm-commits] [llvm] r47469 - /llvm/trunk/include/llvm/CodeGen/SelectionDAGNodes.h

Dan Gohman gohman at apple.com
Thu Feb 21 18:28:56 PST 2008


On Feb 21, 2008, at 5:54 PM, Chris Lattner wrote:

>
> +  const SDOperand &getBasePtr() const {
>     return getOperand(getOpcode() == ISD::LOAD ? 1 : 2);
>   }
> -  const SDOperand getOffset() const {
> -    return getOperand(getOpcode() == ISD::LOAD ? 2 : 3);
> -  }

Why remove LSBaseSDNode::getOffset? Admittedly it wasn't used in
many places, but it allowed the offset assertion check to live in
LSBaseSDNode instead of being in both Load and Store.

Dan




More information about the llvm-commits mailing list