[PATCH] D85074: [WebAssembly] Use "signed char" instead of "char" in SIMD intrinsics.

Dan Gohman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Aug 1 07:53:36 PDT 2020


sunfish created this revision.
sunfish added a reviewer: tlively.
Herald added subscribers: jgravelle-google, sbc100, dschuff.
Herald added a project: clang.
sunfish requested review of this revision.
Herald added a subscriber: aheejin.

This allows people to use `int8_t` instead of `char`, -funsigned-char,
and generally decouples SIMD from the specialness of `char`.

And it makes intrinsics like `__builtin_wasm_add_saturate_s_i8x16`
and `__builtin_wasm_add_saturate_u_i8x16` use signed and unsigned
element types, respectively.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D85074

Files:
  clang/include/clang/Basic/BuiltinsWebAssembly.def
  clang/lib/Headers/wasm_simd128.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D85074.282397.patch
Type: text/x-patch
Size: 10337 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200801/9607af80/attachment.bin>


More information about the cfe-commits mailing list