[PATCH] D113289: LICM: Hoist LOAD without STORE

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 17 14:39:51 PST 2021


lebedev.ri added a comment.

In D113289#3135476 <https://reviews.llvm.org/D113289#3135476>, @reames wrote:

> This needs much clearer motivation - specifically a clear explanation of what you're trying to achieve and why.  Please consider that motivation blocking for further review.

FWIW, i think the general idea behind this is *really* important, "partial" mem2reg is still a win ; D113520 <https://reviews.llvm.org/D113520> is somewhat related.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113289



More information about the llvm-commits mailing list