Page MenuHomeFreeBSD
Differential D20003

Build libclang_rt/profile on all clang-supported architectures
ClosedPublic
Actions

Authored by ngie on Apr 22 2019, 5:57 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Jun 8, 11:28 PM
Unknown Object (File)
Sat, Jun 7, 8:04 PM
Unknown Object (File)
May 28 2025, 2:53 PM
Unknown Object (File)
May 12 2025, 6:18 PM
Unknown Object (File)
May 11 2025, 2:24 PM
Unknown Object (File)
Apr 29 2025, 7:56 PM
Unknown Object (File)
Apr 28 2025, 6:56 AM
Unknown Object (File)
Apr 28 2025, 2:00 AM
View All 75 Files
Subscribers
imp

Details

Reviewers
dim
emaste
jtl
Commits
rS346578: Build libclang_rt/profile on all clang-supported architectures
Summary

There's no reason why a special case needs to be added specifically for amd64,
arm, and i386, as the code is written in machine architecture agnostic C/C++.

This will make it possible for all supporting clang architectures to produce
runtime coverage with --coverage.

MFC after: 2 weeks

Test Plan

"make tinderbox"

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ngie created this revision.Apr 22 2019, 5:57 AM
Herald added a subscriber: imp. · View Herald TranscriptApr 22 2019, 5:57 AM
Harbormaster completed remote builds in B23809: Diff 56469.Apr 22 2019, 5:57 AM
ngie retitled this revision from Build libclang_rt/profile on all clang-supported architectures <<Replace this line with your revision title> to Build libclang_rt/profile on all clang-supported architectures.Apr 22 2019, 5:57 AM
ngie planned changes to this revision.Apr 22 2019, 7:56 AM

This moved SUBDIR+= profile inside another .if block :/...

dim accepted this revision.Apr 22 2019, 7:12 PM

IIRC in the past I tried this on other architectures, and got errors, but if you've ran universe or tinderbox builds with this, I'm fine with it.

ngie updated this revision to Diff 56508.Apr 22 2019, 7:18 PM

Move the SUBDIR+= profile outside the architecture dependent blocks

Tested with "make universe", including the relevant change (^/projects/runtime-coverage-v2@r346532).

Harbormaster completed remote builds in B23830: Diff 56508.Apr 22 2019, 7:18 PM
This revision was not accepted when it landed; it landed in state Needs Review.Apr 22 2019, 7:21 PM
Closed by commit rS346578: Build libclang_rt/profile on all clang-supported architectures (authored by ngie). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
lib/
libclang_rt/
3 lines

Diff 56509

View Options

head/lib/libclang_rt/Makefile

Loading...