[PATCH] D8688: Update MergedLoadStoreMotion to use MemorySSA

Geoff Berry via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 20 12:34:06 PDT 2016


gberry added a comment.

Would it make sense to split out the DFS numbering code motion barrier code into a separate change?


================
Comment at: lib/Transforms/Scalar/MergedLoadStoreMotion.cpp:977
@@ +976,3 @@
+///
+/// \brief True when the store is not downwards explosed to block \p End
+
----------------
Typo: "explosed".
Also, there is no 'End' parameter as referenced in the comment.


http://reviews.llvm.org/D8688





More information about the llvm-commits mailing list