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.

Sign up for GitHub

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

Jump to bottom

Warning options need to be updated when compiling with clang #1

Open
razvand opened this issue Jan 28, 2021 · 3 comments · May be fixed by #10
Open

Warning options need to be updated when compiling with clang #1

razvand opened this issue Jan 28, 2021 · 3 comments · May be fixed by #10
Labels
kind/hackathon Hackathon challenge

Comments

@razvand
Copy link
Contributor

razvand commented Jan 28, 2021

When using clang as the compiler, warning options issue ... warnings:

warning: unknown warning option '-Wno-unused-but-set-variable'; did you mean '-Wno-unused-const-variable'? [-Wunknown-warning-option]
warning: unknown warning option '-Wno-maybe-uninitialized'; did you mean '-Wno-uninitialized'? [-Wunknown-warning-option]
@razvand razvand added the kind/hackathon Hackathon challenge label Sep 10, 2022
@razvand
Copy link
Contributor Author

razvand commented Sep 10, 2022

Hackathon points: 20

@razvand razvand moved this to Todo in Hackathons Mar 31, 2023
@FotisKouzinos
Copy link

working on it

@razvand razvand moved this from Todo to In Progress in Hackathons Mar 31, 2023
@razvand
Copy link
Contributor Author

razvand commented Mar 31, 2023

See this page for info on using clang. Search for clang: https://github.com/mariasfiraiala/scs-work/blob/master/unikraft-scs/unikraft-scs-for-complex-apps.md

iAmGroute pushed a commit to iAmGroute/lib-compiler-rt that referenced this issue Mar 31, 2023
@ntua-el20014 @iAmGroute
Makefile.uk: Don't add unsupported warnings when built with clang
11aa76e 
GitHub-Fixes: unikraft#1

Co-authored-by: Nikos Oikonomou <[email protected]>
Signed-off-by: Nikos Oikonomou <[email protected]>
Signed-off-by: Groute <[email protected]>
iAmGroute pushed a commit to iAmGroute/lib-compiler-rt that referenced this issue Mar 31, 2023
@ntua-el20014 @iAmGroute
Makefile.uk: Don't add unsupported warnings when built with clang
6bbb7a2 
GitHub-Fixes: unikraft#1

Co-authored-by: Nikos Oikonomou <[email protected]>
Signed-off-by: Nikos Oikonomou <[email protected]>
Signed-off-by: Groute <[email protected]>
@iAmGroute iAmGroute linked a pull request Mar 31, 2023 that will close this issue
Open
iAmGroute pushed a commit to iAmGroute/lib-compiler-rt that referenced this issue May 10, 2023
@ntua-el20014 @iAmGroute
Makefile.uk: Don't add unsupported warnings when built with clang
6567bec 
Clang (12.0.0) does not recognize the warnings: no-unused-but-set-variable, no-maybe-uninitialized, no-builtin-declaration-mismatch.
It would print "unknown warning option" warning messages during compilation.
These should be enabled only when compiling with GCC (tested with 11.3.0).

GitHub-Fixes: unikraft#1

Co-authored-by: Nikos Oikonomou <[email protected]>
Signed-off-by: Nikos Oikonomou <[email protected]>
Signed-off-by: Groute <[email protected]>
iAmGroute pushed a commit to iAmGroute/lib-compiler-rt that referenced this issue Jun 20, 2023
@ntua-el20014 @iAmGroute
Makefile.uk: Fix unsupported warnings when built with clang
684a841 
Clang (12.0.0) does not recognize the warnings:
  no-unused-but-set-variable
  no-maybe-uninitialized
  no-builtin-declaration-mismatch
It would print "unknown warning option" warning messages.
These should be enabled only with GCC (tested with 11.3.0).

GitHub-Fixes: unikraft#1

Co-authored-by: Nikos Oikonomou <[email protected]>
Signed-off-by: Nikos Oikonomou <[email protected]>
Signed-off-by: Groute <[email protected]>
iAmGroute pushed a commit to iAmGroute/lib-compiler-rt that referenced this issue Jun 20, 2023
@ntua-el20014 @iAmGroute
Makefile.uk: Fix unsupported warnings when built with clang
0a8cc6e 
Clang (12.0.0) does not recognize the warnings:
  no-unused-but-set-variable
  no-maybe-uninitialized
  no-builtin-declaration-mismatch
It would print "unknown warning option" warning messages.
These should be enabled only with GCC (tested with 11.3.0).

GitHub-Fixes: unikraft#1

Co-authored-by: Nikos Oikonomou <[email protected]>
Signed-off-by: Nikos Oikonomou <[email protected]>
Signed-off-by: Groute <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
kind/hackathon Hackathon challenge
Projects
Hackathons
Status: In Progress
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Makefile.uk: Don't add unsupported warnings when built with clang iAmGroute/lib-compiler-rt
2 participants
@razvand @FotisKouzinos