[PATCH] D113289: LICM: Hoist LOAD without STORE

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 19 11:09:59 PST 2021


efriedma added a comment.

The new summary describing the optimization is a bit confusing to try to parse.  I'd suggest splitting into three paragraphs:

1. What LLVM currently does on the testcase.
2. How the current patch changes the code generation, and why it's an improvement.
3. Potential future directions (fallow-store-data-races etc.)


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

https://reviews.llvm.org/D113289



More information about the llvm-commits mailing list