<html><head><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">This must be code from an older patch.<div><br></div><div>The latest patch has this:</div><div><div>@@ -3223,10 +3232,12 @@ LoopVectorizationCostModel::getInstructionCost(Instruction *I, unsigned VF) {</div><div> // TODO: We need to estimate the cost of intrinsic calls.</div><div> switch (I->getOpcode()) {</div><div> case Instruction::GetElementPtr:</div><div>- // We mark this instruction as zero-cost because scalar GEPs are usually</div><div>- // lowered to the intruction addressing mode. At the moment we don't</div><div>- // generate vector geps.</div><div>+ // We mark this instruction as zero-cost because the cost of GEPs in</div><div>+ // vectorized code depends on whether the corresponding memory instruction</div><div>+ // is scalarized or not. Therefore, we handle GEPs with the memory</div><div>+ // instruction cost.</div><div> return 0;</div><div>+</div><div><br></div><div>I attached the latest patch again. It has to be rebase against master (due to the memory cost refactoring)</div><div><br></div><div></div></div></body></html>