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

[travis] Run clad with clang which has asserts enabled. #157

Closed
wants to merge 5 commits into from

Conversation

vgvassilev
Copy link
Owner

This gives us a great deal of coverage. Clang has well-developed assert system
which often can detect malformed AST. This is very essential for clad as until
today I would need to run that by hand, locally on every pull request.

@vgvassilev vgvassilev force-pushed the enable-rel-with-asserts-osx branch 4 times, most recently from 4286ebc to 8f097d1 Compare December 2, 2019 05:21
@vgvassilev vgvassilev force-pushed the enable-rel-with-asserts-osx branch 3 times, most recently from 83d3b03 to 27b5efd Compare January 17, 2021 12:30
@codecov
Copy link

codecov bot commented Jan 17, 2021

Codecov Report

Merging #157 (3e90e80) into master (144edba) will increase coverage by 1.21%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #157      +/-   ##
==========================================
+ Coverage   82.67%   83.88%   +1.21%     
==========================================
  Files          13       13              
  Lines        2615     2644      +29     
==========================================
+ Hits         2162     2218      +56     
+ Misses        453      426      -27     
Impacted Files Coverage Δ
include/clad/Differentiator/Compatibility.h 68.75% <0.00%> (-7.73%) ⬇️
lib/Differentiator/DiffPlanner.cpp 95.34% <0.00%> (-0.07%) ⬇️
tools/ClangPlugin.h 37.77% <0.00%> (ø)
include/clad/Differentiator/DiffPlanner.h 100.00% <0.00%> (ø)
include/clad/Differentiator/DerivativeBuilder.h 99.04% <0.00%> (+0.05%) ⬆️
lib/Differentiator/DerivativeBuilder.cpp 91.19% <0.00%> (+1.14%) ⬆️
tools/ClangPlugin.cpp 81.65% <0.00%> (+8.57%) ⬆️
Impacted Files Coverage Δ
include/clad/Differentiator/Compatibility.h 68.75% <0.00%> (-7.73%) ⬇️
lib/Differentiator/DiffPlanner.cpp 95.34% <0.00%> (-0.07%) ⬇️
tools/ClangPlugin.h 37.77% <0.00%> (ø)
include/clad/Differentiator/DiffPlanner.h 100.00% <0.00%> (ø)
include/clad/Differentiator/DerivativeBuilder.h 99.04% <0.00%> (+0.05%) ⬆️
lib/Differentiator/DerivativeBuilder.cpp 91.19% <0.00%> (+1.14%) ⬆️
tools/ClangPlugin.cpp 81.65% <0.00%> (+8.57%) ⬆️

@vgvassilev vgvassilev force-pushed the enable-rel-with-asserts-osx branch from 22ff5bf to f0fe389 Compare January 18, 2021 07:26
This gives us a great deal of coverage. Clang has well-developed assert system
which often can detect malformed AST. This is very essential for clad as until
today I would need to run that by hand, locally on every pull request.

At the moment we rely on the fact that llvmlab has such clang builds but
unfortunately only for OSX.
@vgvassilev vgvassilev force-pushed the enable-rel-with-asserts-osx branch 4 times, most recently from de1b0c1 to 3e90e80 Compare January 18, 2021 13:08
@vgvassilev vgvassilev force-pushed the enable-rel-with-asserts-osx branch from 3e90e80 to 47a44ef Compare January 18, 2021 13:23
@vgvassilev vgvassilev force-pushed the master branch 8 times, most recently from 4da9693 to a5ebb8c Compare April 28, 2021 21:21
@vgvassilev vgvassilev added this to the v1.4 milestone Nov 4, 2023
@vgvassilev
Copy link
Owner Author

Resolved by #662

@vgvassilev vgvassilev closed this Nov 30, 2023
@vgvassilev vgvassilev deleted the enable-rel-with-asserts-osx branch November 30, 2023 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant