sunfish added inline comments.

================
Comment at: lib/Basic/Targets.cpp:7010
@@ +7009,3 @@
+  { #ID, TYPE, ATTRS, HEADER, ALL_LANGUAGES, nullptr },
+#include "clang/Basic/BuiltinsWebAssembly.def"
+};
----------------
WebAssembly32TargetInfo is subclassed in LLVM code, and getTargetDefines is 
overridden.

================
Comment at: test/Preprocessor/init.c:8478
@@ +8477,3 @@
+// WEBASSEMBLY32:#define __GCC_ATOMIC_LONG_LOCK_FREE 1{{$}}
+// WEBASSEMBLY32:#define __GCC_ATOMIC_POINTER_LOCK_FREE 1{{$}}
+// WEBASSEMBLY32:#define __GCC_ATOMIC_SHORT_LOCK_FREE 1{{$}}
----------------
Ok, I've set MaxAtomicInlineWidth to 0 for now so that we don't block the rest 
of the patch on this issue.

N4509 is just about exposing the same information via a different interface.


Repository:
  rL LLVM

http://reviews.llvm.org/D12002



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to