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

[Daniel Neilson via Phabricator via llvm-commits]

dneilson added inline comments.


================
Comment at: include/llvm/Analysis/TargetTransformInfoImpl.h:429
+    // in RuntimeLibcalls.h
+    return 4;
+  }
----------------
reames wrote:
> Why 4?  0 would seem like a safer default.
Good point, zero would seem safer...  I put in 4 because my original intent was to put in getRegisterBitWidth()/8 & the default reg bit width is 32 -- I couldn't do that because the call here would get bound to the getRegisterBitWidth() in this base class, and would never see the platform-specific value.


https://reviews.llvm.org/D33243