This is an archive of the discontinued LLVM Phabricator instance.

Mark PR44037 tests as XFAIL on AArch64 Linux dwo
ClosedPublic

Authored by rovka on Nov 18 2019, 2:52 AM.

Details

Summary

These tests are failing with various assertion failures, but they all
throw the following error message first:

error: a.out 0x0000002d: adding range [0x14-0x24) which has a base that
is less than the function's low PC 0x40060c.

See llvm.org/pr44037.

Diff Detail

Event Timeline

rovka created this revision.Nov 18 2019, 2:52 AM
rovka updated this revision to Diff 229794.Nov 18 2019, 3:40 AM
labath resigned from this revision.Nov 18 2019, 3:46 AM

Looks fine to me, but I am not running aarch64 tests. You might want to check of @omjavaid is running them and whether he's ok with that. Otherwise, you can just manage the decorators yourself without any special reviews.

omjavaid accepted this revision.Nov 18 2019, 3:29 PM

Thanks rovka.

LGTM

This revision is now accepted and ready to land.Nov 18 2019, 3:29 PM
rovka added a subscriber: labath.Nov 19 2019, 12:48 AM

Looks fine to me, but I am not running aarch64 tests. You might want to check of @omjavaid is running them and whether he's ok with that. Otherwise, you can just manage the decorators yourself without any special reviews.

Actually, we have a buildbot on the silent master running them:
http://lab.llvm.org:8014/builders/lldb-aarch64-ubuntu

I'm currently just triaging those failures so we can move the bot to the proper master at some point.

Looks fine to me, but I am not running aarch64 tests. You might want to check of @omjavaid is running them and whether he's ok with that. Otherwise, you can just manage the decorators yourself without any special reviews.

Actually, we have a buildbot on the silent master running them:
http://lab.llvm.org:8014/builders/lldb-aarch64-ubuntu

I'm currently just triaging those failures so we can move the bot to the proper master at some point.

I know that. AFAIK @omjavaid, is running that, which is why I suggested to synchronize with him. It would be awesome if we would have an arm bot stable enough to run on master.

This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptNov 19 2019, 1:55 AM