-
Notifications
You must be signed in to change notification settings - Fork 200
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
Add clang builds for embedded TLS adapters #573
Conversation
… point dockes include clang too
@@ -253,6 +253,10 @@ jobs: | |||
name: 'sdk-c--ubuntu-18' | |||
displayName: 'linux_mbed' | |||
steps: | |||
- script: | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
go up to the _PREVIEW_VSTS_DOCKER_IMAGE
variable and make it aziotbld/linux-c-ubuntu-mbed
@@ -287,6 +291,10 @@ jobs: | |||
name: 'sdk-c--ubuntu-18' | |||
displayName: 'wolfssl' | |||
steps: | |||
- script: | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this one aziotbld/linux-c-ubuntu-wolfssl
@@ -327,6 +335,10 @@ jobs: | |||
name: 'sdk-c--ubuntu-18' | |||
displayName: 'bearssl' | |||
steps: | |||
- script: | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
and this one aziotbld/linux-c-ubuntu-bearssl
Our TLS stacks that are primarily used on embedded or low-end Linux devices - wolfssl, bear, and mbed - are currently compiled and tested against gcc only. This is generally fine as gcc is the most common embedded compiler.
However... gcc misses certain warnings condition, especially around uninitialized variables. (See StackOverflow discussion here - I was easily able to reproduce this on gcc even with -Wall and any optimization setting I tried.)
clang does not miss this class of errors. So add a clang build to these stacks. This build only checks compilation step and does not rerun tests gcc currently handles.