[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