[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