[PATCH] D64164: [clang][ARM] Fix msvc arm{64} builtins to use int on LLP64 systems.

JF Bastien via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 3 16:05:53 PDT 2019


jfb accepted this revision.
jfb added inline comments.
This revision is now accepted and ready to land.


================
Comment at: test/CodeGen/ms-intrinsics-other.c:212
+  return _InterlockedAdd(Addend, Value);
+}
+
----------------
Looks like only `cmpxchg` preserves `volatile`? That's not your patch, but seems bad 😖 


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D64164/new/

https://reviews.llvm.org/D64164





More information about the cfe-commits mailing list