Skip to content

Commit ceb32ee

Browse files
committed
Cleanup after recent changes
1 parent 84f042a commit ceb32ee

File tree

3 files changed

+7
-6
lines changed

3 files changed

+7
-6
lines changed

.github/workflows/c-coverage.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ jobs:
5050
- name: Publish coverage-report
5151
uses: JamesIves/github-pages-deploy-action@v4
5252
with:
53-
folder: llvm-cov-report
53+
folder: coverage-report
5454
repository-name: '' # TODO Destination
5555
token: ${{ secrets.COVERAGE_DEPLOY_TOKEN }} # TODO: Use an organization-level token
5656
single-commit: true

Makefile.pre.in

+5-3
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,6 @@ abs_builddir= @abs_builddir@
3737
CC= @CC@
3838
CXX= @CXX@
3939
LINKCC= @LINKCC@
40-
CC_NAME=@CC_NAME@
4140
AR= @AR@
4241
READELF= @READELF@
4342
SOABI= @SOABI@
@@ -661,8 +660,9 @@ bolt-opt: @PREBOLT_RULE@
661660
rm -f $(BUILDPYTHON).bolt_inst
662661
mv $(BUILDPYTHON).bolt $(BUILDPYTHON)
663662

663+
# Support generating coverage reports
664664
.PHONY=coverage-report coverage coverage-generate-lcov coverage-generate-profdata
665-
coverage-report:
665+
coverage-report: regen-token regen-frozen
666666
@ # build with coverage info
667667
$(MAKE) coverage
668668
@ # run tests, ignore failures
@@ -679,7 +679,9 @@ coverage-report:
679679
coverage:
680680
@echo "Building with support for coverage checking:"
681681
$(MAKE) clean
682-
$(MAKE) @DEF_MAKE_RULE@ CC="$(COVERAGE_CC)" CFLAGS="$(CFLAGS) $(COVERAGE_CFLAGS)" LDFLAGS="$(LDFLAGS) $(COVERAGE_LDFLAGS)"
682+
$(MAKE) @DEF_MAKE_RULE@ CC="$(COVERAGE_CC)" \
683+
CFLAGS="$(CFLAGS) $(COVERAGE_CFLAGS)" \
684+
LDFLAGS="$(LDFLAGS) $(COVERAGE_LDFLAGS)"
683685

684686
coverage-generate-lcov:
685687
@echo "Creating Coverage HTML report with LCOV:"
Original file line numberDiff line numberDiff line change
@@ -1,2 +1 @@
1-
The ``coverage-report`` Makefile target will now automatically use ``llvm-cov`` to generate a coverage report when using ``clang``.
2-
This provides more details about branch coverage and subexpressions than the existing ``gcc`` and ``lcov`` based ``coverage-report``.
1+
The ``coverage-report`` Makefile now supports both the ``gcc/lcov`` and ``clang/llvm-profdata`` stacks to generate coverage reports, and will select the correct one based on the compiler in use.

0 commit comments

Comments
 (0)