[PATCH] D57854: [InstCombine] Optimize `atomicrmw <op>, 0` into `load atomic` when possible
JF Bastien via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Feb 6 16:47:04 PST 2019
jfb accepted this revision.
jfb added a subscriber: reames.
jfb added a comment.
This revision is now accepted and ready to land.
One fix then this LGTM
================
Comment at: lib/Transforms/InstCombine/InstCombineAtomicRMW.cpp:39
+ Ordering != AtomicOrdering::Unordered &&
+ "AtomicRMWs don't make sense with Unordered or NotAtomic");
+ if (Ordering != AtomicOrdering::Acquire &&
----------------
Agreed NotAtomic is bonkers (`AtomicRMWInst::Init` disallows it), but Unordered might be sensible, @reames would know.
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D57854/new/
https://reviews.llvm.org/D57854
More information about the llvm-commits
mailing list