This is an archive of the discontinued LLVM Phabricator instance.

WebAssembly: handle more than int32 argument/return
ClosedPublic

Authored by jfb on Jul 31 2015, 5:01 PM.

Details

Summary

Also test 64-bit integers, except shifts for now which are broken because isel dislikes the 32-bit truncate that precedes them.

Diff Detail

Repository
rL LLVM

Event Timeline

jfb updated this revision to Diff 31179.Jul 31 2015, 5:01 PM
jfb retitled this revision from to WebAssembly: handle more than int32 argument/return.
jfb updated this object.
jfb added a reviewer: sunfish.
jfb added a subscriber: llvm-commits.
sunfish accepted this revision.Jul 31 2015, 5:17 PM
sunfish edited edge metadata.
sunfish added inline comments.
test/CodeGen/WebAssembly/integer64.ll
113 ↗(On Diff #31179)

The solution here is to implement getScalarShiftAmountTy for WebAssembly.

This revision is now accepted and ready to land.Jul 31 2015, 5:17 PM
jfb updated this revision to Diff 31185.Jul 31 2015, 9:37 PM
jfb edited edge metadata.
  • Mention getScalarShiftAmountTy.
This revision was automatically updated to reflect the committed changes.
llvm/trunk/lib/Target/WebAssembly/WebAssemblyInstrInfo.td