This is an archive of the discontinued LLVM Phabricator instance.

[WebAssembly] saturating arithmetic builtins
ClosedPublic

Authored by tlively on Oct 3 2018, 6:36 PM.

Diff Detail

Event Timeline

tlively created this revision.Oct 3 2018, 6:36 PM
tlively updated this revision to Diff 168224.Oct 3 2018, 8:52 PM
  • Rebase
craig.topper added inline comments.
include/clang/Basic/BuiltinsWebAssembly.def
59

Don't copy the "V:128:" part from X86. You just need the "nc". The V:128: is a special thing I'm working on for our AVX512 frequency issues.

tlively added inline comments.Oct 4 2018, 2:33 PM
include/clang/Basic/BuiltinsWebAssembly.def
59

ok, sounds good

tlively updated this revision to Diff 168388.Oct 4 2018, 3:04 PM
  • Remove V:128: attributes
aheejin accepted this revision.Oct 4 2018, 5:01 PM
This revision is now accepted and ready to land.Oct 4 2018, 5:01 PM
This revision was automatically updated to reflect the committed changes.