[PATCH] D108517: [MergeICmps] Allow sinking past non-load/store

[Clement Courbet via Phabricator via llvm-commits]

courbet accepted this revision.
courbet added a comment.
This revision is now accepted and ready to land.

Sounds reasonable.

I guess the only caveat that we can only reorder a load/store with non-volatile ones. Removing the check for simplicity of the sinked load/stores looks fine because we are checking that merged loads are simple (=> non-volatile) anyway, so we can indeed reorder  the sinked loads with merged loads.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108517