Differential D49886

Fix build with WITH_CLANG_BOOTSTRAP and WITHOUT_CLANG
ClosedPublic
Actions

Authored by dim on Apr 18 2025, 10:56 AM.

Tags

None

Referenced Files

	F150878564: D49886.diff
	Sat, Apr 4, 4:20 PM

	F150861006: D49886.diff
	Sat, Apr 4, 1:42 PM

	Unknown Object (File)
	Fri, Mar 6, 12:32 PM

	Unknown Object (File)
	Fri, Mar 6, 6:49 AM

	Unknown Object (File)
	Mar 2 2026, 1:59 AM

	Unknown Object (File)
	Feb 27 2026, 11:33 AM

	Unknown Object (File)
	Feb 27 2026, 4:18 AM

	Unknown Object (File)
	Jan 31 2026, 4:18 AM

View All 78 Files

Subscribers

guest-patmaddox

Details

Reviewers

emaste
avg
jhb
imp

Commits

rG5195ff613fca: Fix build with WITH_CLANG_BOOTSTRAP and WITHOUT_CLANG
rG3f68c3e9c138: Fix build with WITH_CLANG_BOOTSTRAP and WITHOUT_CLANG
rGea231471d024: Fix build with WITH_CLANG_BOOTSTRAP and WITHOUT_CLANG

Summary

When WITH_CLANG_BOOTSTRAP and WITHOUT_CLANG are both set, the
cross-tools stage does not build a cross clang binary. This is because
the Makefile in usr.bin/clang checks for WITHOUT_CLANG, and skips
building the binary.

To fix this, ensure that WITH_CLANG is set for the cross-tools phase
whenever WITH_CLANG_BOOTSTRAP is set. While here, skip using the
Makefile in usr.bin/clang, and directly use the Makefile in
usr.bin/clang/clang instead.

Reported by: avg
PR: 286154
MFC after: 1 week

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dim requested review of this revision.Apr 18 2025, 10:56 AM

dim created this revision.

Harbormaster completed remote builds in B63575: Diff 153826.Apr 18 2025, 10:56 AM

LGTM.

Thank you!

This revision is now accepted and ready to land.Apr 18 2025, 1:32 PM

emaste accepted this revision.Apr 18 2025, 1:56 PM

Closed by commit rGea231471d024: Fix build with WITH_CLANG_BOOTSTRAP and WITHOUT_CLANG (authored by dim). · Explain WhyApr 18 2025, 6:33 PM

This revision was automatically updated to reflect the committed changes.

dim added a commit: rGea231471d024: Fix build with WITH_CLANG_BOOTSTRAP and WITHOUT_CLANG.

guest-patmaddox added a subscriber: guest-patmaddox.Apr 18 2025, 9:57 PM

dim added a commit: rG3f68c3e9c138: Fix build with WITH_CLANG_BOOTSTRAP and WITHOUT_CLANG.Apr 23 2025, 7:38 PM

dim added a commit: rG5195ff613fca: Fix build with WITH_CLANG_BOOTSTRAP and WITHOUT_CLANG.

Revision Contents
Changeset List

Path

Size

3 lines

Diff 153880

Makefile.inc1

Loading...