mstorsjo added a comment.
In https://reviews.llvm.org/D37134#868497, @EricWF wrote:
> Ironically I think parts of this change should be moved into
> `merge_archives.py`.
>
> > the ar used in merge_archives.py might not be the right one for the target
>
> OK, then we should modify `merge_archives
EricWF requested changes to this revision.
EricWF added a comment.
Ironically I think parts of this change should be moved into
`merge_archives.py`.
> the ar used in merge_archives.py might not be the right one for the target
OK, then we should modify `merge_archives.py` to support optionally s
mstorsjo added a comment.
Ping @EricWF
https://reviews.llvm.org/D37134
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
compnerd requested changes to this revision.
compnerd added a comment.
This revision now requires changes to proceed.
If we handle this in CMake, this will be unnecessary.
https://reviews.llvm.org/D37134
___
cfe-commits mailing list
cfe-commits@list
mstorsjo added a comment.
Ping
https://reviews.llvm.org/D37134
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mstorsjo created this revision.
Herald added a subscriber: mgorny.
In cross build setups, the ar used in merge_archives.py might not be the right
one for the target (and the invocation doesn't include the 's' modifier for
updating the archive index); make sure to rebuild the index for the static
