-
Notifications
You must be signed in to change notification settings - Fork 43
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
enable travis to use IPO and add a test using libc++ #20
enable travis to use IPO and add a test using libc++ #20
Conversation
the toolchains are installen in /usr/local/bin while other toolchains exist in /usr/bin and ccache provides only part of them in /usr/lib/ccache. CMake then seems to mess up detection of additional tools, this is apparent as the C compiler is not the matching one. Also, output the CmakeCache.txt for diagnostic purpose
becaue I cant figure out how to ignore this configuration as failure =)
last commit for me today, if clang + libc++ still doesnt builds, then allow it as failure. |
I think I get the importance of applying LTO to be better informed about software's correctness. Introducing it might take a considerable amount of time, as I want to know enough about it with respect to CMake, compilers and Travis to reach an easy-to-follow implementation. |
I can build your tests with clang + LTO locally (with libc++ on top if d70b03c is applied), thats not the issue here. ie look at the gcc-7 output:
with clang this is even more of a mess. likely you need to tell CMake the full paths to C/C++ compilers and tools (AR at least). |
b157ba9
to
f283029
Compare
Not sure if this fixes your failures but when setting |
Git conflicts. This can be closed because GitHub Actions have replaced Travis CI. |
I dont know anything about travis, hope this works correctly.
see #18