[PATCH] D132960: [InstCombine] Transform small unaligned memcmp calls used in zero equality tests

[Clement Courbet via Phabricator via llvm-commits]

courbet added a comment.

In D132960#3759691 <https://reviews.llvm.org/D132960#3759691>, @nikic wrote:

> Agree with @efriedma, I don't think we want to introduce unaligned loads without some TTI driven cost modelling.
>
> I believe the original plan was to move the MergeICmps and ExpandMemCmp passes from the backend into the (late) optimization pipeline, which would allow follow-on optimizations to apply. I believe this change also landed at some point, but was reverted (due to some sanitizer interaction possibly?) and nobody bothered bringing it back.

If you're interested in trying it again: https://reviews.llvm.org/D60318


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D132960/new/

https://reviews.llvm.org/D132960