This is an archive of the discontinued LLVM Phabricator instance.

[llvm-objcopy][test] Add cmp after copy
ClosedPublic

Authored by keith on May 12 2022, 10:30 AM.

Details

Summary

All of the other tests here either check that the copy fails, or that
the resulting binary is the same, it seems like this check was omitted
for the universal object case.

Diff Detail

Event Timeline

keith created this revision.May 12 2022, 10:30 AM
keith requested review of this revision.May 12 2022, 10:30 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 12 2022, 10:30 AM
MaskRay accepted this revision.May 12 2022, 11:00 AM

Looks great!

This revision is now accepted and ready to land.May 12 2022, 11:00 AM
This revision was landed with ongoing or failed builds.May 16 2022, 4:05 PM
This revision was automatically updated to reflect the committed changes.

This breaks darwin bot: https://green.lab.llvm.org/green/job/clang-stage1-RA/29421/

Can you take a look or revert if you can't address it immediately? Thanks!

We're seeing an internal failure after this change as well :( Great catch, but it looks like it exposed some existing brokenness.

Yea it passes on my M1 machine, I will test on Intel and see if that's the difference, I have reverted since so hopefully everyone is back to green and I'll debug async and post here with the new fix.