This is an archive of the discontinued LLVM Phabricator instance.

[WebAssembly] Renumber and LEB128-encode SIMD opcodes
ClosedPublic

Authored by tlively on Nov 5 2018, 4:33 PM.

Diff Detail

Event Timeline

tlively created this revision.Nov 5 2018, 4:33 PM
aheejin added inline comments.Nov 5 2018, 5:16 PM
lib/Target/WebAssembly/WebAssemblyInstrSIMD.td
40

Is there any reason the section orders in this file have all changed? If possible keeping them the same would be helpful to view the diff to figure out what part has changed in which way.

tlively updated this revision to Diff 172696.Nov 5 2018, 6:03 PM
  • Restore original section order
tlively marked an inline comment as done.Nov 5 2018, 6:03 PM
aheejin added a comment.EditedNov 6 2018, 3:00 PM

This CL does not only do renumbering but also do some of refactoring by unifying classes into unaries and binaries. Ideally they can be splitted, but if we want to do them herer together maybe we can add that part of explanation to the CL/commit description.

lib/Target/WebAssembly/WebAssemblyInstrSIMD.td
683–684

Can we use SIMDUnary for this?

tlively updated this revision to Diff 173214.Nov 8 2018, 1:09 PM
  • Renumber again
tlively updated this revision to Diff 173228.Nov 8 2018, 2:32 PM
  • Use SIMDUnary for NOT
tlively marked an inline comment as done.Nov 8 2018, 2:33 PM
aheejin accepted this revision.Nov 8 2018, 2:45 PM
This revision is now accepted and ready to land.Nov 8 2018, 2:45 PM
This revision was automatically updated to reflect the committed changes.