[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