[PATCH] D34629: Add isElementAtomic query method to MemInstrinsic class.

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 30 12:19:14 PDT 2017


efriedma added a comment.

Oh, I see...

What's the advantage of "I->isElementAtomic()" over "isa<ElementUnorderedAtomicMemCpyInst>(I)"?

Not sure it makes sense to make ElementUnorderedAtomicMemCpyInst inherit from MemCpyInst; it's likely to be confusing that a MemCpyInst might not really be a memcpy.


https://reviews.llvm.org/D34629





More information about the llvm-commits mailing list