[PATCH] D116375: [X86] Use bit test instructions to optimize some logic atomic operations

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: llvm/lib/Target/X86/X86AtomicANDHoist.cpp:61
+      Changed = true;
+      And->moveAfter(AI);
+    }
----------------
It might not be safe to move the And if the other operand isn't a constant. There's no guarantee that the atomic instruction occurs after the source of the other operand.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D116375/new/

https://reviews.llvm.org/D116375