This is an archive of the discontinued LLVM Phabricator instance.

[llvm-lipo] Relax the check of the specified input file architecture
ClosedPublic

Authored by alexander-shaposhnikov on Oct 1 2019, 10:19 PM.

Details

Summary

For -arch cctools' lipo only compares the cputypes,
e.g. see the function "check_arch" in
https://opensource.apple.com/source/cctools/cctools-921/misc/lipo.c.auto.html .
This diff adjusts the behavior of llvm-lipo accordingly.

Test plan: make check-all

Diff Detail

Repository
rL LLVM

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptOct 1 2019, 10:19 PM
plotfi accepted this revision.Oct 2 2019, 12:02 AM

Make sure to clang format the new code. I think this looks ok.

tools/llvm-lipo/llvm-lipo.cpp
448 ↗(On Diff #222754)

Is this comment clang-formated>

This revision is now accepted and ready to land.Oct 2 2019, 12:02 AM
compnerd accepted this revision.Oct 3 2019, 9:25 AM