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">hfinkel@anl.gov</a> <<a href="mailto:hfinkel@anl.gov">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>