Skip to content

Change a few functions related to runtime library paths to use the correct architecture specific paths #15560

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

lanza
Copy link
Contributor

@lanza lanza commented Mar 28, 2018

Adjust the runtimeLibPath for the GenerixUnix toolchain to use the
architecture specific path.

Modify the runtimeLibraryPath in CompilerInvocation.cpp to use the
architecture specific version for non-Darwin platforms.

Modify a few tests to agree with these changes.

@lanza
Copy link
Contributor Author

lanza commented Mar 28, 2018

@compnerd please test

@compnerd
Copy link
Member

@swift-ci please test and merge

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - b1159c80318d8ff5b6c841f8d4ec903c5163cdff

@compnerd
Copy link
Member

@lanza seems like something here is the cause: https://ci.swift.org/job/swift-PR-Linux/3968/consoleFull#5841992143122a513-f36a-4c87-8ed7-cbc36a1ec144

/usr/bin/ld.gold: error: cannot find -lswiftImageInspectionShared

@lanza
Copy link
Contributor Author

lanza commented Apr 26, 2018

@compnerd this works on top of the changes in #16191

@lanza
Copy link
Contributor Author

lanza commented Apr 28, 2018

@compnerd Done.

@lanza
Copy link
Contributor Author

lanza commented Jul 12, 2018

@swift-ci please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 9415e5e8553c95433748c4ab1cdd768d4c3481d9

…rrect architecture specific path

Adjust the runtimeLibPath for the GenerixUnix toolchain to use the
architecture specific path.

Modify the `runtimeLibraryPath` in CompilerInvocation.cpp to use the
architecture specific version for non-Darwin platforms.

Modify a few tests to agree with these changes.
@lanza
Copy link
Contributor Author

lanza commented Jul 13, 2018

Hmm this failed due to a typo in swiftpm that has since been fixed.

@lanza
Copy link
Contributor Author

lanza commented Jul 13, 2018

@swift-ci please test linux platform

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 9415e5e8553c95433748c4ab1cdd768d4c3481d9

@lanza lanza closed this Jul 13, 2018
@lanza lanza deleted the arch-dirs branch February 11, 2019 19:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants