This is an archive of the discontinued LLVM Phabricator instance.

test/DebugInfo/Generic/missing-abstract-variable.ll fails on arm
AbandonedPublic

Authored by weimingz on Nov 18 2015, 10:52 AM.

Details

Summary

Does anyone else have the same issue? Should we XFail it?

Diff Detail

Event Timeline

weimingz updated this revision to Diff 40534.Nov 18 2015, 10:52 AM
weimingz retitled this revision from to test/DebugInfo/Generic/missing-abstract-variable.ll fails on arm.
weimingz updated this object.
weimingz added a subscriber: Restricted Project.
mcrosier edited reviewers, added: dexonsmith, aprantl; removed: mcrosier.Nov 18 2015, 10:57 AM
mcrosier added a subscriber: mcrosier.
rengolin requested changes to this revision.Nov 18 2015, 1:50 PM
rengolin added a reviewer: rengolin.

Hi Weiming,

The buildbots are green for a good number of years, so they're pretty good baseline for you to see if the problem happens upstream or not, meaning XFAILing tests on ARM or AArch64 are no longer accepted as a solution.

It may be that whatever revision you have in your tree broke that test, or that your environment is slightly different, but the bots are authoritative (we have to have some baseline). They have a list of commits you can look at to make sure your tree is as up-to-date as the bots, and if not, you should either update it or look down the waterfall to see if the particular commit you have fails or not.

Regardless, if the bots are currently green, XFAILing a test will break them. If it's not, fixing the problem is the best solution.

cheers,
--renato

This revision now requires changes to proceed.Nov 18 2015, 1:50 PM
weimingz abandoned this revision.Nov 18 2015, 2:15 PM

Hi Weiming,

The buildbots are green for a good number of years, so they're pretty good baseline for you to see if the problem happens upstream or not, meaning XFAILing tests on ARM or AArch64 are no longer accepted as a solution.

It may be that whatever revision you have in your tree broke that test, or that your environment is slightly different, but the bots are authoritative (we have to have some baseline). They have a list of commits you can look at to make sure your tree is as up-to-date as the bots, and if not, you should either update it or look down the waterfall to see if the particular commit you have fails or not.

Regardless, if the bots are currently green, XFAILing a test will break them. If it's not, fixing the problem is the best solution.

cheers,
--renato

Hi Renato,

Thanks. I'll look down to see why it fails in my local machine.

Thanks,
weiming