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

Reid Kleckner via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 11 13:32:54 PDT 2018


rnk added a comment.

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.


Repository:
  rC Clang

https://reviews.llvm.org/D47672





More information about the cfe-commits mailing list