Ericsson / clang

Cross Translation Unit analysis capability for Clang Static Analyzer. (Fork of official clang at http://llvm.org/git/clang)

http://clang.llvm.org/

Other

15 stars 10 forks source link

ctu-clang8 patches #667

Closed balazske closed 5 years ago

balazske commented 5 years ago

One test (Misc/warning-flags.c) fails, probably because the new warning in #663 . Warning count should be updated to 75.

martong commented 5 years ago

One test (Misc/warning-flags.c) fails, probably because the new warning in #663 . Warning count should be updated to 75.

Could you please fix the warning count too?

balazske commented 5 years ago

The test says that we should add a -W flag for the new warnings, is this not applicable for ASTImporter case (even then it is good to have some documentation about it)?

martong commented 5 years ago

The test says that we should add a -W flag for the new warnings, is this not applicable for ASTImporter case (even then it is good to have some documentation about it)?

We can do that later if needed. But in previous branches (ctu-clang7, ctu-clang6) we have not added the -W flag for the new warnings, instead we changed this count.

balazske commented 5 years ago

659 #660 #663 #664 are still missing, can I add these (in this PR)?

martong commented 5 years ago

659 #660 #663 #664 are still missing, can I add these (in this PR)?

Well, I merged it already, could you please add them in a new PR?