[PATCH] Add two helper functions: isAtLeastAcquire, isAtLeastRelease

Robin Morisset morisset at google.com
Mon Aug 11 10:50:26 PDT 2014


If they were for generic instructions, yes (but generic instructions don't have a MemoryOrdering).
Here they are for loads/stores, which cannot have the acq_rel memory ordering (as said in a comment a few lines before each of my changes).
If this patch is later extended to RMW instructions, then yes acq_rel will have to be taken into account.

http://reviews.llvm.org/D4844






More information about the llvm-commits mailing list