Add a comment, per Bruno's CR.
authorNadav Rotem <nadav.rotem@intel.com>
Thu, 11 Aug 2011 17:05:47 +0000 (17:05 +0000)
committerNadav Rotem <nadav.rotem@intel.com>
Thu, 11 Aug 2011 17:05:47 +0000 (17:05 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@137313 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/X86/X86ISelLowering.cpp

index 94faae5b556be4389f183cb6a0db5a8f69f472d1..666ce3c07e96f18bcf84f72d36dcecf3718d535f 100644 (file)
@@ -12593,6 +12593,9 @@ static SDValue PerformSTORECombine(SDNode *N, SelectionDAG &DAG,
   const TargetLowering &TLI = DAG.getTargetLoweringInfo();
 
   // If we are saving a concatination of two XMM registers, perform two stores.
+  // This is better in Sandy Bridge cause one 256-bit mem op is done via two
+  // 128-bit ones. If in the future the cost becomes only one memory access the
+  // first version would be better.
   if (VT.getSizeInBits() == 256 &&
     StoredVal.getNode()->getOpcode() == ISD::CONCAT_VECTORS &&
     StoredVal.getNumOperands() == 2) {