Use toolprefix for archiver ar when building libcompcert.a#380
Closed
MSoegtropIMC wants to merge 1 commit intoAbsInt:masterfrom
Closed
Use toolprefix for archiver ar when building libcompcert.a#380MSoegtropIMC wants to merge 1 commit intoAbsInt:masterfrom
MSoegtropIMC wants to merge 1 commit intoAbsInt:masterfrom
Conversation
Contributor
|
Well spotted, thank you for reporting this issue and for the proposed fix. I'll fix it slightly differently so that the correct archiver is selected also when the Diab toolchain is used. (The archiver is called Testing is in progress. |
Contributor
Author
|
Perfect, thanks! I hope it is agreeable to include your fix as patch in the CompCert 3.8 opam package. Since this shouldn't change anything on platforms where it does work without this, I would still call it version 3.8. |
Contributor
|
See commit ca2ebae . You are most welcome to use it as a patch for the CompCert OPAM package. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
For compiling a cross CompCert, e.g. a 64 bit CompCert on 32 bit Cygwin, it is sufficient to set the toolprefix option. But trying to build an executable with ccomp fails with an error that libcompcert.a doesn't have an index. Using the toolprefix also for the archiver when building libcompcert.a fixes this issue.
This fix is required to build a working 64 bit CompCert for a 32 bit Windows Coq. Some researchers prefer the 32 bit version of Coq even on 64 bit Windows because it is faster (needs less memory and memoy bandwidth).
Please note that I plan to include this change in the CompCert 3.8 opam packages.