[PATCH] D113289: LICM: Hoist LOAD without STORE

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 16 11:06:15 PST 2021


lebedev.ri added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/LICM.cpp:120-125
+// Hoist the load without hoisting the store. Some targets (such as aarch64)
+// will gain benefits out of this, since there will be better register
+// promotion.
+static cl::opt<bool> EnableLoadHoistOnly(
+    "licm-load-hoisting-only", cl::Hidden, cl::init(false),
+    cl::desc("Enable hoisting loads without hoisting the stores"));
----------------
This is really ambiguous to me. Is this talking about
*only* about hoisting loads, and preventing hoisting stores,
or hoisting loads even if we can't hoist the store?


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