[PATCH] D33243: [Atomics][LoopIdiom] Recognize unordered atomic memcpy

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 30 15:21:24 PDT 2017


reames accepted this revision.
reames added a comment.

LGTM

Anna, since Daniel doesn't yet have commit access would you mind landing this?  I would do it myself, but can't commit the time to watch for any problems after commit.



================
Comment at: lib/Transforms/Scalar/LoopIdiomRecognize.cpp:362
+    return LegalStoreKind::None;
+  // We only want simple or unordered-atomic stores (note: unordered => simple)
+  if (!SI->isUnordered())
----------------
I don't understand the note?  Maybe just remove


https://reviews.llvm.org/D33243





More information about the llvm-commits mailing list