[PATCH] D47672: [Headers] Add _Interlocked*_HLEAcquire/_HLERelease

Tue Jun 12 16:09:16 PDT 2018

ethanhs added a comment.

In https://reviews.llvm.org/D47672#1128863, @rnk wrote:

> In https://reviews.llvm.org/D47672#1128308, @hans wrote:
>
> > It sounds like adding proper support for HLE prefixes is a largeish project.
> >
> > ctopper, rnk: Do you think it would be worth adding inline asm versions (with the xacquire/release prefixes) of these intrinsics in the meantime? It would inhibit optimizations but be better than the current state of not having the intrinsics at all.
>
>
> Yeah, let's do that. I'm a lot more comfortable ignoring bugs about missed optimizations with fancy intrinsics than bugs that say the intrinsic doesn't do what it says it does.

I'm afraid I've never actually written inline asm in C, but if no one else wants to take this, I'm willing to try to figure it out. I'd like to be able to use these in my CPython port :)

Repository:
  rC Clang

https://reviews.llvm.org/D47672