[PATCH] D32762: [Atomic] Remove IsStore/IsLoad in the interface, and pass the instruction instead. NFC.

Sanjoy Das via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun May 7 13:22:51 PDT 2017


sanjoy added a comment.

Random drive by comment.



================
Comment at: llvm/include/llvm/IR/Instruction.h:460
+  /// Return true if this atomic instruction loads from memory.
+  bool isAtomicLoad() const;
+
----------------
Would it be better to call this `hasAtomicLoad`?  Since the instructions here do not *solely* atomic-load.


https://reviews.llvm.org/D32762





More information about the llvm-commits mailing list