[llvm] r258830 - Reassociate: Reprocess RedoInsts after each inst

[Hans Wennborg via llvm-commits]

Bisection of https://llvm.org/bugs/show_bug.cgi?id=28367 points to this commit.

Aditya: can you investigate?

Thanks,
Hans

On Tue, Jan 26, 2016 at 10:42 AM, Aditya Nandakumar via llvm-commits
<llvm-commits at lists.llvm.org> wrote:
> Author: aditya_nandakumar
> Date: Tue Jan 26 12:42:36 2016
> New Revision: 258830
>
> URL: http://llvm.org/viewvc/llvm-project?rev=258830&view=rev
> Log:
> Reassociate: Reprocess RedoInsts after each inst
>
> Previously the RedoInsts was processed at the end of the block.
> However it was possible that it left behind some instructions that
> were not canonicalized.
> This should guarantee that any previous instruction in the basic
> block is canonicalized before we process a new instruction.
>
> Added:
>     llvm/trunk/test/Transforms/Reassociate/prev_insts_canonicalized.ll
> Modified:
>     llvm/trunk/lib/Transforms/Scalar/Reassociate.cpp
>     llvm/trunk/test/Transforms/Reassociate/reassoc-intermediate-fnegs.ll
>     llvm/trunk/test/Transforms/Reassociate/xor_reassoc.ll