From: Heejin Ahn Date: Mon, 20 Aug 2018 23:49:34 +0000 (+0000) Subject: [WebAssembly] Revert type of wake count in atomic.wake to i32 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=43e71a0b50e85ee4b755ca43e554099dba14decc;p=clang [WebAssembly] Revert type of wake count in atomic.wake to i32 Summary: We decided to revert this from i64 to i32 in Nov 28 CG meeting. Fixes PR38632. Reviewers: dschuff Subscribers: sbc100, jgravelle-google, sunfish, jfb, cfe-commits Differential Revision: https://reviews.llvm.org/D51013 git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@340235 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/include/clang/Basic/BuiltinsWebAssembly.def b/include/clang/Basic/BuiltinsWebAssembly.def index 45ce07292e..d987af6492 100644 --- a/include/clang/Basic/BuiltinsWebAssembly.def +++ b/include/clang/Basic/BuiltinsWebAssembly.def @@ -37,6 +37,6 @@ BUILTIN(__builtin_wasm_rethrow, "v", "r") // Atomic wait and notify. BUILTIN(__builtin_wasm_atomic_wait_i32, "ii*iLLi", "n") BUILTIN(__builtin_wasm_atomic_wait_i64, "iLLi*LLiLLi", "n") -BUILTIN(__builtin_wasm_atomic_notify, "ULLii*LLi", "n") +BUILTIN(__builtin_wasm_atomic_notify, "Uii*i", "n") #undef BUILTIN diff --git a/test/CodeGen/builtins-wasm.c b/test/CodeGen/builtins-wasm.c index d1c5c303d4..1eb5222884 100644 --- a/test/CodeGen/builtins-wasm.c +++ b/test/CodeGen/builtins-wasm.c @@ -63,8 +63,8 @@ int f9(long long *addr, long long expected, long long timeout) { // WEBASSEMBLY64: call i32 @llvm.wasm.atomic.wait.i64(i64* %{{.*}}, i64 %{{.*}}, i64 %{{.*}}) } -unsigned long long f10(int *addr, long long count) { +unsigned int f10(int *addr, int count) { return __builtin_wasm_atomic_notify(addr, count); -// WEBASSEMBLY32: call i64 @llvm.wasm.atomic.notify(i32* %{{.*}}, i64 %{{.*}}) -// WEBASSEMBLY64: call i64 @llvm.wasm.atomic.notify(i32* %{{.*}}, i64 %{{.*}}) +// WEBASSEMBLY32: call i32 @llvm.wasm.atomic.notify(i32* %{{.*}}, i32 %{{.*}}) +// WEBASSEMBLY64: call i32 @llvm.wasm.atomic.notify(i32* %{{.*}}, i32 %{{.*}}) }