This is an archive of the discontinued LLVM Phabricator instance.

[WebAssembly] Massive instruction renaming
ClosedPublic

Authored by tlively on Jan 4 2019, 3:31 PM.

Details

Diff Detail

Repository
rL LLVM

Event Timeline

tlively created this revision.Jan 4 2019, 3:31 PM
aardappel accepted this revision.Jan 7 2019, 9:21 AM

I'm fine with this, though it is not obvious to me from the associated github issue that consensus was reached.. can you point me in the right direction?

This revision is now accepted and ready to land.Jan 7 2019, 9:21 AM

I'm fine with this, though it is not obvious to me from the associated github issue that consensus was reached.. can you point me in the right direction?

https://github.com/WebAssembly/spec/issues/884#issuecomment-447518428 I don't think there was a formal consensus but the changes were landed in the spec.

This revision was automatically updated to reflect the committed changes.