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

Quentin Colombet via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 16 13:43:18 PST 2015


qcolombet added a subscriber: qcolombet.
qcolombet added a comment.

Hi Mehdi,

Please consider merging the test files. They all have the same test line and as far as I know you are fixing the source of one crash, not eight.
Also please:

- Add check lines.
- Use opt -instnamer to get rid of the %[num] variables.

My 2c.

Thanks,
-Quentin


http://reviews.llvm.org/D6995





More information about the llvm-commits mailing list