[PATCH] D52858: [WebAssembly] saturating arithmetic builtins

2018-10-04 Thread Thomas Lively via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL343836: [WebAssembly] saturating arithmetic builtins (authored by tlively, committed by ). Herald added a subscriber: llvm-commits. Repository: rL LLVM https://reviews.llvm.org/D52858 Files: cfe/tru

[PATCH] D52858: [WebAssembly] saturating arithmetic builtins

2018-10-04 Thread Thomas Lively via Phabricator via cfe-commits
tlively updated this revision to Diff 168388. tlively added a comment. - Remove V:128: attributes Repository: rC Clang https://reviews.llvm.org/D52858 Files: include/clang/Basic/BuiltinsWebAssembly.def lib/CodeGen/CGBuiltin.cpp test/CodeGen/builtins-wasm.c Index: test/CodeGen/builtins

[PATCH] D52858: [WebAssembly] saturating arithmetic builtins

2018-10-04 Thread Thomas Lively via Phabricator via cfe-commits
tlively added inline comments. Comment at: include/clang/Basic/BuiltinsWebAssembly.def:59 +BUILTIN(__builtin_wasm_add_saturate_s_i8x16, "V16cV16cV16c", "ncV:128:") +BUILTIN(__builtin_wasm_add_saturate_u_i8x16, "V16cV16cV16c", "ncV:128:") craig.topper wrote: > D

[PATCH] D52858: [WebAssembly] saturating arithmetic builtins

2018-10-03 Thread Craig Topper via Phabricator via cfe-commits
craig.topper added inline comments. Comment at: include/clang/Basic/BuiltinsWebAssembly.def:59 +BUILTIN(__builtin_wasm_add_saturate_s_i8x16, "V16cV16cV16c", "ncV:128:") +BUILTIN(__builtin_wasm_add_saturate_u_i8x16, "V16cV16cV16c", "ncV:128:") Don't copy the "V:

[PATCH] D52858: [WebAssembly] saturating arithmetic builtins

2018-10-03 Thread Thomas Lively via Phabricator via cfe-commits
tlively updated this revision to Diff 168224. tlively added a comment. - Rebase Repository: rC Clang https://reviews.llvm.org/D52858 Files: include/clang/Basic/BuiltinsWebAssembly.def lib/CodeGen/CGBuiltin.cpp test/CodeGen/builtins-wasm.c Index: test/CodeGen/builtins-wasm.c ==

[PATCH] D52858: [WebAssembly] saturating arithmetic builtins

2018-10-03 Thread Thomas Lively via Phabricator via cfe-commits
tlively created this revision. tlively added reviewers: aheejin, dschuff. Herald added subscribers: cfe-commits, kristina, sunfish, jgravelle-google, sbc100. Depends on https://reviews.llvm.org/D52856. Repository: rC Clang https://reviews.llvm.org/D52858 Files: include/clang/Basic/Builtin