[PATCH] D24593: Standford/Bubble sort code restructure

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 6 10:00:08 PDT 2016


mehdi_amini added a comment.

In https://reviews.llvm.org/D24593#563106, @evstupac wrote:

> > This is not flaky.
>
> Maybe it is not flaky. But what I'm trying to say that now the test is not ok to measure compiler performance.


I was digging in my archive, because in the past "someone" (didn't remember who) provided great explanations as of why a test case wasn't great for a particular microarchitecture, and I wanted to point at it as an example of the level of information I'd expected from someone to convince about what's going on beyond simply a claim that "this is bad luck with the predictor".

I just found it: https://llvm.org/bugs/show_bug.cgi?id=5615 (See the description, and the attached PPT).
(I noticed when finding it that the name seems familiar...)


https://reviews.llvm.org/D24593





More information about the llvm-commits mailing list