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

[Serguei Katkov via Phabricator via llvm-commits]

skatkov added inline comments.


================
Comment at: lib/Transforms/Scalar/LoopIdiomRecognize.cpp:991
+  // previous checks.
+  if (SI->isAtomic() || LI->isAtomic()) {
+    // element.unordered.atomic is limited to 16-byte element-size because
----------------
I would prefer to revert the if statement.
if SI and LI are not atomic then memcpy otherwise your code.

It is simpler to follow if the short case is handled first.


https://reviews.llvm.org/D33243