Skip to content
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

CI: enable LLVM in Jenkins #6600

Closed
wants to merge 5 commits into from

Conversation

smlng
Copy link
Member

@smlng smlng commented Feb 14, 2017

this PR enables all tests on native using LLVM toolchain, it depends on #6598 #6609.

@smlng smlng added Area: CI Area: Continuous Integration of RIOT components Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation labels Feb 14, 2017
@smlng smlng requested a review from cgundogan February 14, 2017 14:13
@smlng smlng added the State: waiting for other PR State: The PR requires another PR to be merged first label Feb 14, 2017
@smlng smlng force-pushed the pr/ci/jenkins/enable_llvm branch from d0b3fa7 to 8324589 Compare February 15, 2017 12:27
@smlng
Copy link
Member Author

smlng commented Feb 15, 2017

@gebart now LLVM fails when compiling tests/unittests/ with:

kernel_init.c:58:18: error: self-comparison always evaluates to true [-Werror,-Wtautological-compare]
    if (LOG_INFO <= LOG_INFO) printf("main(): This is RIOT! (Version: " "buildtest" ")\n");
                 ^
1 error generated.

you see this error too?`

@jnohlgard
Copy link
Member

jnohlgard commented Feb 15, 2017 via email

@jnohlgard
Copy link
Member

jnohlgard commented Feb 15, 2017 via email

@smlng
Copy link
Member Author

smlng commented Feb 15, 2017

@gebart Jenkins uses clang: clang version 3.8.0-2ubuntu4 (tags/RELEASE_380/final), so its a bit older than yours (3.9.x something, IIRC?).

@smlng
Copy link
Member Author

smlng commented Feb 15, 2017

needs #6612 #6744

@smlng smlng force-pushed the pr/ci/jenkins/enable_llvm branch 2 times, most recently from 086c537 to 2ad5a45 Compare February 15, 2017 14:11
@smlng
Copy link
Member Author

smlng commented Feb 15, 2017

and #6606 merged

@smlng smlng force-pushed the pr/ci/jenkins/enable_llvm branch 5 times, most recently from fce9928 to ac9b21f Compare February 15, 2017 16:18
@smlng smlng force-pushed the pr/ci/jenkins/enable_llvm branch from ac9b21f to 3d6a79f Compare February 17, 2017 08:58
@kYc0o
Copy link
Contributor

kYc0o commented Mar 3, 2017

Can we try to get this merged before #6673 ?

@smlng
Copy link
Member Author

smlng commented Mar 3, 2017

nope, there are more parts missing.

@smlng smlng force-pushed the pr/ci/jenkins/enable_llvm branch 2 times, most recently from c522e5d to c8c82aa Compare March 13, 2017 21:20
@smlng smlng force-pushed the pr/ci/jenkins/enable_llvm branch from c8c82aa to 9875f06 Compare March 15, 2017 10:40
@smlng
Copy link
Member Author

smlng commented Mar 16, 2017

separated commits into extra PR #6752

@smlng smlng added the CI: needs squashing Commits in this PR need to be squashed; If set, CI systems will mark this PR as unmergable label Mar 16, 2017
@smlng smlng force-pushed the pr/ci/jenkins/enable_llvm branch 2 times, most recently from 661ed34 to fafbde0 Compare March 17, 2017 08:05
@smlng smlng force-pushed the pr/ci/jenkins/enable_llvm branch from 80e1d72 to fd29269 Compare March 20, 2017 15:10
@smlng
Copy link
Member Author

smlng commented May 12, 2017

closing as memo, as we go for Murdock 2 instead Jenkins - at least for the time being

@smlng smlng closed this May 12, 2017
@smlng smlng added the State: archived State: The PR has been archived for possible future re-adaptation label May 12, 2017
@smlng smlng deleted the pr/ci/jenkins/enable_llvm branch May 31, 2017 18:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: CI Area: Continuous Integration of RIOT components CI: needs squashing Commits in this PR need to be squashed; If set, CI systems will mark this PR as unmergable State: archived State: The PR has been archived for possible future re-adaptation State: waiting for other PR State: The PR requires another PR to be merged first Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants