[PATCH] [clang-cl] Added _InterlockedCompareExchangePointer to Intrin.h

[David Ziman]

================
Comment at: lib/Headers/Intrin.h:783
@@ +782,3 @@
+__faststorefence(void) {
+	__asm__ volatile ("lock orq $0,(%rsp)");
+}
----------------
Reid Kleckner wrote:
> I think this needs to have a memory constraint to avoid reordering non-volatile loads and stores across it.
This instruction reads a memory address, or's it with the constant 0, and writes it back to the same location with the lock prefix. That's read-modify-write with lock prefix, and I believe the AMD and Intel manuals state that this should be enough. I can provide references if you like. It's also the same instruction emitted by MSVC.

================
Comment at: lib/Headers/Intrin.h:783
@@ +782,3 @@
+__faststorefence(void) {
+	__asm__ volatile ("lock orq $0,(%rsp)");
+}
----------------
David Ziman wrote:
> Reid Kleckner wrote:
> > I think this needs to have a memory constraint to avoid reordering non-volatile loads and stores across it.
> This instruction reads a memory address, or's it with the constant 0, and writes it back to the same location with the lock prefix. That's read-modify-write with lock prefix, and I believe the AMD and Intel manuals state that this should be enough. I can provide references if you like. It's also the same instruction emitted by MSVC.
Alternatively, we could use sfence here. Are you a also looking to prevent the compiler from re-ordering this instruction? Will it do that?

http://reviews.llvm.org/D2600