Differential D40674

bsd.sys.mk: Re-enable -Wunused-but-set-variable for clang 13+.
ClosedPublic
Actions

Authored by jhb on Jun 20 2023, 4:34 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Jan 27, 9:51 AM

	Unknown Object (File)
	Mon, Jan 27, 8:23 AM

	Unknown Object (File)
	Sat, Jan 25, 12:49 PM

	Unknown Object (File)
	Jan 10 2025, 6:20 PM

	Unknown Object (File)
	Jan 10 2025, 6:19 PM

	Unknown Object (File)
	Jan 9 2025, 12:54 AM

	Unknown Object (File)
	Nov 5 2024, 8:26 AM

	Unknown Object (File)
	Oct 29 2024, 11:46 PM

View All 43 Files

Subscribers

Details

Reviewers

dim
emaste

Commits

rGd036adc4e913: bsd.sys.mk: Re-enable -Wunused-but-set-variable for clang 13+.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jhb created this revision.Jun 20 2023, 4:34 PM

Herald added subscribers: emaste, imp, bdrewery. · View Herald TranscriptJun 20 2023, 4:34 PM

jhb requested review of this revision.Jun 20 2023, 4:34 PM

Harbormaster completed remote builds in B52209: Diff 123549.Jun 20 2023, 4:34 PM

jhb added a parent revision: D40673: rtld-elf: Mark tls_init_align __unused in free_tls for Variant I TLS..Jun 20 2023, 4:34 PM

I started on just trying to bring GCC up to par with clang (only disabling it for WARNS < 3) but accidentally had enabled it always for GCC when I got to the end, so I went ahead and just enabled it for clang as well and got make tinderbox working. The result is this stack/series (as well as about 20 commits or so I just pushed to main that were trivial enough to not warrant review IMO).

emaste accepted this revision.Jun 20 2023, 4:57 PM

This revision is now accepted and ready to land.Jun 20 2023, 4:57 PM

Great, let's prevent more of these getting into the tree! :)

Closed by commit rGd036adc4e913: bsd.sys.mk: Re-enable -Wunused-but-set-variable for clang 13+. (authored by jhb). · Explain WhyJun 28 2023, 7:28 PM

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rGd036adc4e913: bsd.sys.mk: Re-enable -Wunused-but-set-variable for clang 13+..

Revision Contents
Changeset List

Path

Size

share/

mk/

3 lines

Diff 123919

share/mk/bsd.sys.mk

Loading...