Hi,<br><br>Don't get me wrong, I'm not trying to torpedo it or anything. It's fine by me to go in. I'd just like to raise awareness that there's a greater problem that this doesn't seem to go towards solving. <br><br>Sorry if I came across like a complainer :<br><br>James<br><div class="gmail_quote">On Wed, 12 Nov 2014 at 16:41 Arnold Schwaighofer <<a href="mailto:aschwaighofer@apple.com">aschwaighofer@apple.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">I see this as an intermediate step that will allow us to catch certain patterns. If performance tests show it is beneficial why not take it?<div><br></div><div>Enabling horizontal reductions is a second step that might subsume this logic.</div></div><div style="word-wrap:break-word"><div><br></div><div><br><div><blockquote type="cite"><div>On Nov 12, 2014, at 8:33 AM, James Molloy <<a href="mailto:james@jamesmolloy.co.uk" target="_blank">james@jamesmolloy.co.uk</a>> wrote:</div><br><div>Hi,<br><br>This seems to me to be a sticking plaster over a bigger problem. This will fix some reductions, but the bigger problem of loops being unrolled and then not optimised still exists. I'd much prefer to see a solution to this problem than just adding more Elastoplast-logic to SLP.<br><br>James<br><div class="gmail_quote">On Wed, 12 Nov 2014 at 16:24 <a href="mailto:hfinkel@anl.gov" target="_blank">hfinkel@anl.gov</a> <<a href="mailto:hfinkel@anl.gov" target="_blank">hfinkel@anl.gov</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">================<br>
Comment at: test/Transforms/SLPVectorizer/<u></u>X86/return.ll:58<br>
@@ +57,3 @@<br>
+<br>
+attributes #0 = { nounwind readonly "less-precise-fpmad"="false" "no-frame-pointer-elim"="<u></u>false" "no-infs-fp-math"="false" "no-nans-fp-math"="false" "stack-protector-buffer-size"=<u></u>"8" "unsafe-fp-math"="false" "use-soft-float"="false" }<br>
+<br>
----------------<br>
Please remove unnecessary attributes.<br>
<br>
================<br>
Comment at: test/Transforms/SLPVectorizer/<u></u>X86/return.ll:62<br>
@@ +61,3 @@<br>
+<br>
+!0 = metadata !{metadata !"clang version 3.6.0 (221134)"}<br>
+!1 = metadata !{metadata !2, metadata !2, i64 0}<br>
----------------<br>
Please remove unnecessary metadata.<br>
<br>
<a href="http://reviews.llvm.org/D6227" target="_blank">http://reviews.llvm.org/D6227</a><br>
<br>
<br>
<br>
______________________________<u></u>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/llvm-commits</a><br>
</blockquote></div>
</div></blockquote></div><br></div></div></blockquote></div>