Page MenuHomeFreeBSD
Differential D14265

Promote llvm-cov to a standalone option
ClosedPublic
Actions

Authored by emaste on Feb 8 2018, 4:37 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Sep 23, 3:15 AM
Unknown Object (File)
Fri, Sep 11, 1:20 PM
Unknown Object (File)
Thu, Sep 10, 11:32 AM
Unknown Object (File)
Mon, Sep 7, 1:05 AM
Unknown Object (File)
Sun, Sep 6, 6:14 PM
Unknown Object (File)
Sun, Sep 6, 1:55 PM
Unknown Object (File)
Sun, Sep 6, 10:16 AM
Unknown Object (File)
Sun, Sep 6, 7:23 AM
View All 66 Files
Subscribers
bdrewery
imp
jhb

Details

Reviewers
dim
imp
Summary

Introduce WITH_/WITHOUT_LLVM_COV to match GCC's WITH_/WITHOUT_GCOV.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

emaste created this revision.Feb 8 2018, 4:37 PM
Herald added a subscriber: bdrewery. · View Herald TranscriptFeb 8 2018, 4:37 PM
emaste added inline comments.Feb 8 2018, 4:38 PM
lib/clang/libllvm/Makefile
667

IMO the extra cost of just compiling this always is small enough to not be worth the extra complexity of compiling it for LLVM_COV in addition to the current SRCS_EXT.

imp accepted this revision.Feb 8 2018, 4:45 PM
This revision is now accepted and ready to land.Feb 8 2018, 4:45 PM
dim accepted this revision.Feb 8 2018, 7:12 PM
emaste added a comment.Nov 5 2018, 3:37 PM

Committed in rS329093 with wrong URL

emaste closed this revision.Nov 5 2018, 3:38 PM

Revision Contents
Changeset List

PathSize
lib/
clang/
libllvm/
2 lines
share/
mk/
2 lines
tools/
build/
options/
4 lines
4 lines
usr.bin/
clang/
6 lines

Diff 39043

View Options

lib/clang/libllvm/Makefile

Loading...
View Options

share/mk/src.opts.mk

Loading...
View Options

tools/build/options/WITHOUT_LLVM_COV

Loading...
View Options

tools/build/options/WITH_LLVM_COV

Loading...
View Options

usr.bin/clang/Makefile

Loading...