[PATCH] D33584: Remove a quadratic behavior in assert-enabled builds

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 25 22:18:49 PDT 2017


davide added a comment.

In https://reviews.llvm.org/D33584#765359, @anemet wrote:

> In https://reviews.llvm.org/D33584#765358, @davide wrote:
>
> > Adding that to the testsuite, if it's not there, would be great.
> >  If you can attach it here (or point a location where I can fetch, I'd like to run it under a profiler :)
>
>
> I can't really share the code as is but should be able to generate something that demonstrates the issue.


Sure.

> I don't think  we want to add it to the LLVM tests though.  I.e. what would be the failure mode?

Maybe not the llvm-test suite, but test-suite [this: https://github.com/llvm-mirror/test-suite] (as it's starting to include testcases to show compile time regressions).


https://reviews.llvm.org/D33584





More information about the llvm-commits mailing list