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

Daniel Neilson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 31 08:07:19 PDT 2017


dneilson marked an inline comment as done.
dneilson added inline comments.


================
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())
----------------
reames wrote:
> I don't understand the note?  Maybe just remove
The test is for 'isUnordered()', and the note aims to clarify that unordered implies (=>) simple.


https://reviews.llvm.org/D33243





More information about the llvm-commits mailing list