This is an archive of the discontinued LLVM Phabricator instance.

[Coroutine] Rename coro-semmetric-transfer.cpp and fix test failure
ClosedPublic

Authored by lxfind on Oct 12 2020, 2:20 PM.

Details

Summary

Some tests start to fail after https://reviews.llvm.org/D89066.
It's because the size of pointers are different on different targets.
Limit the target in the command so there is no confusion.
Also noticed I had typo in the test name.
Adding disable-llvm-passes option to make the test more stable as well.

Diff Detail

Event Timeline

lxfind created this revision.Oct 12 2020, 2:20 PM
Herald added a project: Restricted Project. · View Herald TranscriptOct 12 2020, 2:20 PM
lxfind requested review of this revision.Oct 12 2020, 2:20 PM
lxfind retitled this revision from [Coroutine] Rename coro-semmetric-transfer.cpp and possibly fix test failure to [Coroutine] Rename coro-semmetric-transfer.cpp and fix test failure.Oct 12 2020, 2:52 PM
wenlei accepted this revision.Oct 12 2020, 3:11 PM
This revision is now accepted and ready to land.Oct 12 2020, 3:11 PM
This revision was landed with ongoing or failed builds.Oct 12 2020, 3:29 PM
This revision was automatically updated to reflect the committed changes.