[PATCH] D33243: [Atomics][LoopIdiom] Recognize unordered atomic memcpy
Serguei Katkov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon May 22 23:18:58 PDT 2017
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
More information about the llvm-commits
mailing list