Added a common about the need for X86ISD::Wrapper.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@26372 91177308-0d34-0410-b5e6-96231b3b80d8

diff --git a/lib/Target/X86/X86ISelLowering.cpp b/lib/Target/X86/X86ISelLowering.cpp
index e2f22ae0a3e5488d92755d773a503e556aa57c27..48e049c6175edd77d6303d6b78fe865ec0ff74c3 100644 (file)
--- a/lib/Target/X86/X86ISelLowering.cpp
+++ b/lib/Target/X86/X86ISelLowering.cpp
@@ -1829,6 +1829,13 @@ SDOperand X86TargetLowering::LowerOperation(SDOperand Op, SelectionDAG &DAG) {
     return DAG.getNode(X86ISD::REP_MOVS, MVT::Other, Chain,
                        DAG.getValueType(AVT), InFlag);
   }
+
+  // ConstantPool, GlobalAddress, and ExternalSymbol are lowered as their
+  // target countpart wrapped in the X86ISD::Wrapper node. Suppose N is
+  // one of the above mentioned nodes. It has to be wrapped because otherwise
+  // Select(N) returns N. So the raw TargetGlobalAddress nodes, etc. can only
+  // be used to form addressing mode. These wrapped nodes will be selected
+  // into MOV32ri.
   case ISD::ConstantPool: {
     ConstantPoolSDNode *CP = cast<ConstantPoolSDNode>(Op);
     SDOperand Result = DAG.getNode(X86ISD::Wrapper, getPointerTy(),