[PATCH] D86864: [MachineSinking] sink more profitable loads

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

Do we need some threshold to limit the analysis?  I'm worried this could get very expensive.



================
Comment at: llvm/lib/CodeGen/MachineSink.cpp:890
+    return true;
+
+  // Does these two blocks pair cache as no store before?
----------------
Are the post-dominance checks necessary?  I'm not sure what invariants you're trying to enforce.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86864