diff options
| author | Thomas Lively <tlively@google.com> | 2018-10-25 19:06:15 +0000 |
|---|---|---|
| committer | Thomas Lively <tlively@google.com> | 2018-10-25 19:06:15 +0000 |
| commit | 535b4df75ab5ff57cda64c53d69acbff49de7ca2 (patch) | |
| tree | 03c8ab8d0de8d3d7ce27dc9fb5bbf30f767d4f77 | |
| parent | 0aad98fd07ae8b554fd75e335a21ccb0b1615c9b (diff) | |
| download | bcm5719-llvm-535b4df75ab5ff57cda64c53d69acbff49de7ca2.tar.gz bcm5719-llvm-535b4df75ab5ff57cda64c53d69acbff49de7ca2.zip | |
[WebAssembly] Lower to target-independent saturating add
Summary: Goes along with D53721.
Reviewers: aheejin, dschuff
Subscribers: sbc100, jgravelle-google, sunfish, cfe-commits
Differential Revision: https://reviews.llvm.org/D53722
llvm-svn: 345300
| -rw-r--r-- | clang/lib/CodeGen/CGBuiltin.cpp | 4 | ||||
| -rw-r--r-- | clang/test/CodeGen/builtins-wasm.c | 8 |
2 files changed, 6 insertions, 6 deletions
diff --git a/clang/lib/CodeGen/CGBuiltin.cpp b/clang/lib/CodeGen/CGBuiltin.cpp index 9ae2df2dd79..3d1cf75952c 100644 --- a/clang/lib/CodeGen/CGBuiltin.cpp +++ b/clang/lib/CodeGen/CGBuiltin.cpp @@ -12614,11 +12614,11 @@ Value *CodeGenFunction::EmitWebAssemblyBuiltinExpr(unsigned BuiltinID, switch (BuiltinID) { case WebAssembly::BI__builtin_wasm_add_saturate_s_i8x16: case WebAssembly::BI__builtin_wasm_add_saturate_s_i16x8: - IntNo = Intrinsic::wasm_add_saturate_signed; + IntNo = Intrinsic::sadd_sat; break; case WebAssembly::BI__builtin_wasm_add_saturate_u_i8x16: case WebAssembly::BI__builtin_wasm_add_saturate_u_i16x8: - IntNo = Intrinsic::wasm_add_saturate_unsigned; + IntNo = Intrinsic::uadd_sat; break; case WebAssembly::BI__builtin_wasm_sub_saturate_s_i8x16: case WebAssembly::BI__builtin_wasm_sub_saturate_s_i16x8: diff --git a/clang/test/CodeGen/builtins-wasm.c b/clang/test/CodeGen/builtins-wasm.c index 1e94a8d8184..56b5d32f3eb 100644 --- a/clang/test/CodeGen/builtins-wasm.c +++ b/clang/test/CodeGen/builtins-wasm.c @@ -223,28 +223,28 @@ f64x2 replace_lane_f64x2(f64x2 v, double x) { i8x16 add_saturate_s_i8x16(i8x16 x, i8x16 y) { return __builtin_wasm_add_saturate_s_i8x16(x, y); - // WEBASSEMBLY: call <16 x i8> @llvm.wasm.add.saturate.signed.v16i8( + // WEBASSEMBLY: call <16 x i8> @llvm.sadd.sat.v16i8( // WEBASSEMBLY-SAME: <16 x i8> %x, <16 x i8> %y) // WEBASSEMBLY-NEXT: ret } i8x16 add_saturate_u_i8x16(i8x16 x, i8x16 y) { return __builtin_wasm_add_saturate_u_i8x16(x, y); - // WEBASSEMBLY: call <16 x i8> @llvm.wasm.add.saturate.unsigned.v16i8( + // WEBASSEMBLY: call <16 x i8> @llvm.uadd.sat.v16i8( // WEBASSEMBLY-SAME: <16 x i8> %x, <16 x i8> %y) // WEBASSEMBLY-NEXT: ret } i16x8 add_saturate_s_i16x8(i16x8 x, i16x8 y) { return __builtin_wasm_add_saturate_s_i16x8(x, y); - // WEBASSEMBLY: call <8 x i16> @llvm.wasm.add.saturate.signed.v8i16( + // WEBASSEMBLY: call <8 x i16> @llvm.sadd.sat.v8i16( // WEBASSEMBLY-SAME: <8 x i16> %x, <8 x i16> %y) // WEBASSEMBLY-NEXT: ret } i16x8 add_saturate_u_i16x8(i16x8 x, i16x8 y) { return __builtin_wasm_add_saturate_u_i16x8(x, y); - // WEBASSEMBLY: call <8 x i16> @llvm.wasm.add.saturate.unsigned.v8i16( + // WEBASSEMBLY: call <8 x i16> @llvm.uadd.sat.v8i16( // WEBASSEMBLY-SAME: <8 x i16> %x, <8 x i16> %y) // WEBASSEMBLY-NEXT: ret } |
