<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">The change looks good to me but we need a small test case for the change. <div><br></div><div>Thanks,</div><div>Nadav<br><div><br><div><div>On Nov 21, 2013, at 1:59 PM, Yi Jiang <<a href="mailto:yjiang@apple.com">yjiang@apple.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><meta http-equiv="Content-Type" content="text/html charset=us-ascii"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">Hi,<div><br></div><div>This patch is about cost model of SLP Vectorizer. Now the extract cost of all external uses of the scalars will be calculated and added up to the total cost. Actually, only once is enough for each scalar even if it has multiple external uses since we usually just generate one extractelement operation. It will expose more SLP vectorization opportunities on Olden/power and we see 11% improvement on that case. Any comments is highly appreciated. </div><div><br></div><div><a href="http://yijiang-mini1.apple.com/perf/v4/nts/2/graph?test.1=3">MultiSource/Benchmarks/Olden/power/power</a> -11.41% 1.6936 1.5004</div><div><br></div><div><br></div><div></div></div>
<span><slpextractcost.patch></span><meta http-equiv="Content-Type" content="text/html charset=us-ascii"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div></div></div>
_______________________________________________<br>llvm-commits mailing list<br><a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits<br></blockquote></div><br></div></div></body></html>