Paths

Table of Contentst

Differential D50466

share/mk: pass SHARED_CFLAGS to PIC and PIE from assembly
ClosedPublic
Actions

Authored by brooks on Thu, May 22, 3:02 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, May 29, 11:34 AM

	Unknown Object (File)
	Wed, May 28, 6:51 PM

	Unknown Object (File)
	Tue, May 27, 11:05 PM

	Unknown Object (File)
	Sun, May 25, 5:34 AM

	Unknown Object (File)
	Sat, May 24, 8:40 AM

	Unknown Object (File)
	Sat, May 24, 2:09 AM

	Unknown Object (File)
	Sat, May 24, 1:15 AM

	Unknown Object (File)
	Sat, May 24, 12:57 AM

View All 9 Files

Subscribers

Details

Reviewers

dg612_cam.ac.uk
jrtc27
jhb
dim
emaste

Commits

rG9c2001036185: share/mk: pass SHARED_CFLAGS to PIC and PIE from assembly

Summary

This mirrors what we do for C code.

PR: 286975

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

brooks created this revision.Thu, May 22, 3:02 PM

Herald added subscribers: emaste, imp, bdrewery. · View Herald TranscriptThu, May 22, 3:02 PM

brooks requested review of this revision.Thu, May 22, 3:02 PM

Harbormaster completed remote builds in B64355: Diff 155868.Thu, May 22, 3:03 PM

brooks added a parent revision: D50465: libsys: Use PICO_CFLAGS to disable syscall stubs.Thu, May 22, 3:03 PM

I wish we had these documented somewhere, so we could include a note that ${SHARED_CFLAGS} (along with ${CFLAGS} and ${PIEO_CFLAGS}/${PICO_CFLAGS} etc.).

I see three SHARED_CFLAGS references: in lib/libc/sys/Makefile.inc (noted in the PR), share/mk/bsd.sanitizer.mk, and lib/libarchive/Makefile.inc. I'm not sure why libarchive's addition of ICONV-related flags are in SHARED_CFLAGS not CFLAGS.

brooks added a parent revision: D50475: rtld: get syscall implementations from libsys.Thu, May 22, 7:05 PM

Drop PICO_CFLAGS, and friends
Document SHARED_CFLAGS

Harbormaster completed remote builds in B64369: Diff 155897.Thu, May 22, 10:33 PM

brooks added a parent revision: D50481: share/mk/bsd.README: refer to the linker, not the loader.Thu, May 22, 10:35 PM

emaste accepted this revision.Thu, May 22, 11:28 PM

This revision is now accepted and ready to land.Thu, May 22, 11:28 PM

jhb accepted this revision.Fri, May 23, 5:39 PM

Closed by commit rG9c2001036185: share/mk: pass SHARED_CFLAGS to PIC and PIE from assembly (authored by brooks). · Explain WhyFri, May 23, 11:49 PM

This revision was automatically updated to reflect the committed changes.

brooks added a commit: rG9c2001036185: share/mk: pass SHARED_CFLAGS to PIC and PIE from assembly.

Revision Contents
Changeset List

Path

Size

share/

mk/

3 lines

bsd.suffixes-extra.mk

10 lines

Diff 155959

share/mk/bsd.README

Loading...

share/mk/bsd.suffixes-extra.mk

Loading...