[PATCH] D45151: [LICM] Hoisting invariant.group loads
Jakub Kuderski via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Apr 3 20:28:25 PDT 2018
kuhar accepted this revision.
kuhar added inline comments.
This revision is now accepted and ready to land.
================
Comment at: lib/Transforms/Scalar/LICM.cpp:522
+
+ // For now we only want to hoist invariant.group loads only if we can keep
+ // the metadata. This is because we don't know yet if it's better to hoist it
----------------
nit: I'd remove the second 'only'
Repository:
rL LLVM
https://reviews.llvm.org/D45151
More information about the llvm-commits
mailing list