This is an archive of the discontinued LLVM Phabricator instance.

[CMake] Add Python script to generate version script symbol exports
ClosedPublic

Authored by andrewng on Oct 17 2022, 10:01 AM.

Details

Summary

Using a Python script instead of the various shell commands means that
it is now possible to cross compile LLVM for Linux on Windows.

Diff Detail

Event Timeline

andrewng created this revision.Oct 17 2022, 10:01 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 17 2022, 10:01 AM
andrewng requested review of this revision.Oct 17 2022, 10:01 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 17 2022, 10:01 AM
llvm/utils/add_llvm_symbol_exports.py
2

please add the copyright section

12

please run black on this file.
Line 7 = 2 spaces
line 11 = 8 spaces

andrewng updated this revision to Diff 468479.Oct 18 2022, 4:27 AM

Update for review comments.

andrewng marked 2 inline comments as done.Oct 18 2022, 4:31 AM
andrewng added inline comments.
llvm/utils/add_llvm_symbol_exports.py
12

Sorry about this. Not sure what happened here but a mix of tabs, spaces and tab widths created a bit of a mess!

sylvestre.ledru accepted this revision.Oct 18 2022, 4:59 AM

LGTM

llvm/utils/add_llvm_symbol_exports.py
12

no worries, we should have automation to catch this in a perfect world...

This revision is now accepted and ready to land.Oct 18 2022, 4:59 AM
This revision was automatically updated to reflect the committed changes.
andrewng marked an inline comment as done.