Allow the kernel to build with a compiler that sets -fno-common.
ClosedPublic
Actions

Authored by brooks on Apr 6 2020, 10:35 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Nov 28, 8:30 AM

	Unknown Object (File)
	Thu, Nov 27, 10:47 PM

	Unknown Object (File)
	Tue, Nov 18, 5:55 AM

	Unknown Object (File)
	Nov 5 2025, 9:44 PM

	Unknown Object (File)
	Nov 5 2025, 2:50 PM

	Unknown Object (File)
	Nov 5 2025, 4:12 AM

	Unknown Object (File)
	Nov 1 2025, 12:34 AM

	Unknown Object (File)
	Oct 20 2025, 10:58 AM

View All 81 Files

Subscribers

None

Details

Reviewers

kevans
jhb
imp

Commits

rS359691: Allow the kernel to build with a compiler that sets -fno-common.

Summary

The mechanism that generates assym.inc and offset.inc depends on the
symbols in question being common. For now, simply force the object files
to be created with -fcommon.

Diff Detail

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 30334
Build 28102: arc lint + arc unit

Event Timeline

brooks created this revision.Apr 6 2020, 10:35 PM

Harbormaster completed remote builds in B30334: Diff 70283.Apr 6 2020, 10:35 PM

I took a brief look at rewriting the awk to work with either -fcommon or -fno-common a lttle bit ago, but that's non-trivial as things don't line up as neatly.

This revision is now accepted and ready to land.Apr 6 2020, 10:38 PM

The values are also quite different if you diff the nm output. I think this whole thing will need to be rethought eventually, but removing -fcommon is probably a long ways off.

Closed by commit rS359691: Allow the kernel to build with a compiler that sets -fno-common. (authored by brooks). · Explain WhyApr 7 2020, 3:32 PM

This revision was automatically updated to reflect the committed changes.

brooks added a commit: rS359691: Allow the kernel to build with a compiler that sets -fno-common..