[PATCH] D42793: [MergeICmps] Enable the MergeICmps Pass by default.

[Sanjay Patel via Phabricator via llvm-commits]

spatel added a comment.

In https://reviews.llvm.org/D42793#996217, @courbet wrote:

> Rebase the tests to show codegen diffs.


Thanks!

Thinking about this a bit more, since MergeICmps and MemCmpExpansion use the same TLI hook, this shouldn't be a problem unless there's already a bug in PPC's enabling of that hook. Given that MemCmpExpansion was originally written with a PPC target in mind, there's not much risk of that.

So I think this patch is fine now, but let's give someone who's closer to current-day PPC metal a chance to reply.


Repository:
  rL LLVM

https://reviews.llvm.org/D42793