[PATCH] D24593: Standford/Bubble sort code restructure
Mehdi AMINI via llvm-commits
llvm-commits at lists.llvm.org
Wed Sep 14 16:14:24 PDT 2016
mehdi_amini added a comment.
I'm not to understand what is "flaky" with this test.
The fact that the compiler is not smart enough seems that we should improve the compiler, not simplify the benchmark.
I'm probably missing the whole point. The link you're providing https://reviews.llvm.org/D18158 contains extensive discussion, can you be more specific where to look (or what to look for)?
================
Comment at: SingleSource/Benchmarks/Stanford/Bubblesort.c:134
@@ -133,1 +133,3 @@
+ biggest = sortlist[1]; littlest = sortlist[1];
+ for ( i = 2; i <= srtelements; i++ ) {
temp = Rand();
----------------
I don't understand why this change? (i.e. it should be at least documented).
https://reviews.llvm.org/D24593
More information about the llvm-commits
mailing list