search

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

SourceLocation import improvements. #574

Closed balazske closed 5 years ago

balazske commented 5 years ago

I could not find out why the ASTMerge/interface test fails in Jenkins. (Test must be tried with sanitizer enabled?)

martong commented 5 years ago

I could not find out why the ASTMerge/interface test fails in Jenkins. (Test must be tried with sanitizer enabled?)

Yes, it failed with a clang which has the sanitizers enabled. But, I think the error would be different in case of memory corruption reported by asan.

martong commented 5 years ago

Does it pass with GCC, if you skip the failing step of the pipeline?

balazske commented 5 years ago

On my computer the tests pass with normal GCC debug and release build (full check-clang). It passes with release sanitizer build using current clang compiler too.

martong commented 5 years ago

I have cleaned up the workspace of the pr job (deleted the build and source dirs), lets see what happens now.

run tests

balazske commented 5 years ago

OK I will split the patch.

balazske commented 5 years ago

Related: #594

balazske commented 5 years ago

Related: #595

balazske commented 5 years ago

Next: #596

balazske commented 5 years ago

Last part: #598

balazske commented 5 years ago

The changes are made in separate PRs, closing this one.