[PATCH] D24149: add testcases for PR30188

Sanjay Patel via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 2 14:11:58 PDT 2016


spatel added a subscriber: spatel.
spatel added a comment.

I thought we frown on multi-pass regression tests, let alone -O2.

Based on the comments in PR30188 ( https://llvm.org/bugs/show_bug.cgi?id=30188 ) - if the goal is to make sure that BinarySearch perf does not regress, is a better solution to add that C source app to test-suite's Benchmarks dir, so we can make sure that the end-to-end perf is what we want?


https://reviews.llvm.org/D24149





More information about the llvm-commits mailing list