This is an archive of the discontinued LLVM Phabricator instance.

[WebAssembly] Trying to fix broken tests caused by r281285.
ClosedPublic

Authored by ioeric on Sep 13 2016, 2:18 AM.

Diff Detail

Event Timeline

ioeric updated this revision to Diff 71122.Sep 13 2016, 2:18 AM
ioeric retitled this revision from to [WebAssembly] Trying to fix broken tests in CodeGen/WebAssembly caused by r281285..
ioeric updated this object.
ioeric added reviewers: bkramer, ddcc, dschuff, sunfish.
ioeric added a subscriber: llvm-commits.
ioeric retitled this revision from [WebAssembly] Trying to fix broken tests in CodeGen/WebAssembly caused by r281285. to [WebAssembly] Trying to fix broken tests caused by r281285..Sep 13 2016, 2:24 AM
djasper accepted this revision.Sep 13 2016, 3:12 AM
djasper edited edge metadata.

This makes sense to me. At any rate, it is only fixing broken tests so it can't make the actual behavior worse. Submit this to get the bots back to green. If necessary, the owners can do further changes later.

This revision is now accepted and ready to land.Sep 13 2016, 3:12 AM
ioeric updated this object.Sep 13 2016, 3:13 AM
ioeric edited edge metadata.
This revision was automatically updated to reflect the committed changes.
dschuff edited edge metadata.Sep 13 2016, 7:47 AM

Thanks!