This is an archive of the discontinued LLVM Phabricator instance.

[WebAssembly] Rename StoreResults to MemIntrinsicResults
ClosedPublic

Authored by aheejin on Dec 26 2018, 2:27 PM.

Details

Summary

StoreResults pass does not optimize store instructions anymore because
store instructions don't return results values anymore. Now this pass is
used solely for memory intrinsics, so update the pass name accordingly
and fix outdated pass descriptions as well.

This patch does not change any meaningful behavior, but not marked as
NFC because it changes a comment check line in a test case.

Diff Detail

Event Timeline

aheejin created this revision.Dec 26 2018, 2:27 PM
aheejin updated this revision to Diff 179528.Dec 26 2018, 2:30 PM
  • wrap to 80 cols in README.txt
This revision is now accepted and ready to land.Jan 8 2019, 11:30 AM
This revision was automatically updated to reflect the committed changes.