[PATCH] D33847: [PartialInlining] Enhance code outliner to sink locals declared outside the outline region

[David Li via Phabricator via llvm-commits]

davidxl added a comment.

Any more comments?



================
Comment at: lib/Transforms/Utils/CodeExtractor.cpp:185-186
+          StoreInst *SI = cast<StoreInst>(&II);
+          if (!SI->isUnordered())
+            return false;
+          MemAddr = SI->getPointerOperand();
----------------
davide wrote:
> Can you add a comment explaining why you skip ordered load/stores?
not necessarily needed. Removed.


https://reviews.llvm.org/D33847