[PATCH] D20379: Codegen: Fix broken assumption in Tail Merge.

Chad Rosier via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 15 08:12:43 PDT 2016


mcrosier requested changes to this revision.
mcrosier added a comment.
This revision now requires changes to proceed.

In http://reviews.llvm.org/D20379#457684, @iteratee wrote:

> OK, the two failing tests are gone, I don't think they were valid to begin with. Are there any other concerns about this change?


In general, we don't want to reduce our test coverage.  Please provide additional details as to why you believe these two tests aren't valid and why it's reasonable to delete them.


http://reviews.llvm.org/D20379





More information about the llvm-commits mailing list