This is an archive of the discontinued LLVM Phabricator instance.

[WebAssembly] Improve readability of load/stores and tests. NFC.
ClosedPublic

Authored by aheejin on Jul 9 2018, 9:59 AM.

Details

Summary
  • Changed variable/function names to be more consistent
  • Improved comments in test files
  • Added more tests
  • Fixed a few typos
  • Misc. cosmetic changes

Diff Detail

Event Timeline

aheejin created this revision.Jul 9 2018, 9:59 AM
aheejin updated this revision to Diff 154640.Jul 9 2018, 10:41 AM
  • Pattern name change: anyext_aload -> sext_aload

This pattern doesn't cover all the sign extension selection (we need a
sext_inreg after that), but it looks less confusing and more consistent with
zext_aload patterns. And we gave enough explanations in comments anyway.

dschuff accepted this revision.Jul 9 2018, 12:01 PM
This revision is now accepted and ready to land.Jul 9 2018, 12:01 PM
This revision was automatically updated to reflect the committed changes.