[PATCH] D6995: Reassociate: reprocess RedoInsts after each instruction

[Duncan P. N. Exon Smith via llvm-commits]

> On 2015-Nov-16, at 13:47, Mehdi AMINI <mehdi.amini at apple.com> wrote:
> 
> joker.eph added a comment.
> 
> Only 3 test are still crashing now, but at the time I wrote the patch, I iterated on a fuzzer and these six tests (not eight) were stressing different patterns and different part of reassociate, so they're absolutely not redundant.

You could use six different functions in a single .ll file, right?

> I'm not sure either what CHECK line to put for a compiler crash non-regression, there are multiple cases in the test suite like this where FileCheck is not involved.

It's best practice to check for an expected output even if it's just
a crasher.  Most of the tests where FileCheck isn't involved are
legacy (and should be fixed if possible).

> 
> 
> http://reviews.llvm.org/D6995
> 
> 
>