This is an archive of the discontinued LLVM Phabricator instance.

[WebAssembly] Update relaxed-simd instruction names
ClosedPublic

Authored by tlively on Nov 17 2022, 5:17 PM.

Details

Summary

Including builtin and intrinsic names. These should be the final names for the
proposal.
https://github.com/WebAssembly/relaxed-simd/blob/main/proposals/relaxed-simd/Overview.md

Diff Detail

Event Timeline

tlively created this revision.Nov 17 2022, 5:17 PM
Herald added a project: Restricted Project. · View Herald TranscriptNov 17 2022, 5:17 PM
Herald added subscribers: pmatos, asb, wingo and 6 others. · View Herald Transcript
tlively requested review of this revision.Nov 17 2022, 5:17 PM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptNov 17 2022, 5:17 PM
maratyszcza added inline comments.Nov 18 2022, 8:10 AM
clang/include/clang/Basic/BuiltinsWebAssembly.def
180

Typo?

tlively added inline comments.Nov 18 2022, 8:22 AM
clang/include/clang/Basic/BuiltinsWebAssembly.def
180

Yep, thanks.

tlively updated this revision to Diff 476490.Nov 18 2022, 8:24 AM
  • Fix type
aheejin accepted this revision.Nov 18 2022, 11:00 AM
This revision is now accepted and ready to land.Nov 18 2022, 11:00 AM
maratyszcza accepted this revision.Nov 18 2022, 8:50 PM

Intrinsic naming LGTM. Not qualified to review the rest.

This revision was automatically updated to reflect the committed changes.