[llvm-commits] Patch for LangRef.html as part of Atomic Operations

[Reid Spencer]

On Fri, 2007-07-20 at 10:44 -0500, Dan Gohman wrote:
> >> +<div class="doc_subsubsection">
> >> +  <a name="atomic_membarrier">'<tt>llvm.atomic.membarrier</tt>'
> >> Intrinsic</a>
> >
> > Consider naming this llvm.atomic.memory.barrier ?
> 
> I know I said earlier that I liked the "atomic" in the names, but
> for membarrier it's a little confusing, since the operation
> is relevent for all memory operations, not just the "atomic" ones.
> 
> How about llvm.membarrier, or llvm.memory.barrier if you like
> Reid's proposed convention?

Yeah, I was thinking the same thing. It doesn't read well. I put
"atomic" in there just for grouping with the other related intrinsics.
But, its probably better to get an accurate name. I'm fine with dropping
the atomic part.

Reid.

> 
> Dan
>