lld: add -z nodump support
AbandonedPublic
Actions

Authored by emaste on Sep 14 2018, 1:53 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Jan 15 2024, 11:59 AM

	Unknown Object (File)
	Dec 20 2023, 12:14 AM

	Unknown Object (File)
	Dec 18 2023, 9:51 PM

	Unknown Object (File)
	Dec 13 2023, 4:28 PM

	Unknown Object (File)
	Nov 30 2023, 10:45 AM

	Unknown Object (File)
	Sep 22 2023, 5:22 PM

	Unknown Object (File)
	Sep 4 2023, 2:09 PM

	Unknown Object (File)
	Aug 15 2023, 12:26 PM

Subscribers

Details

Reviewers

dim
markj

Summary

-z nodump sets the DF_1_NODUMP flag, marking the object as not dumpable by the runtime linker.

FreeBSD does not support DF_1_NODUMP, so the flag will be set in the object but ignored by rtld.

PR:             230603
Reported by:    jbeich
Sponsored by:   The FreeBSD Foundation

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

emaste created this revision.Sep 14 2018, 1:53 PM

Upstream review: https://reviews.llvm.org/D52096

markj accepted this revision.Sep 14 2018, 2:58 PM

This revision is now accepted and ready to land.Sep 14 2018, 2:58 PM

LGTM. I assume this is going upstream too?

Yeah I posted an upstream review in https://reviews.llvm.org/D52096 but based on what Mark found on the mailing list I'm more inclined to say we don't want to support this misfeature and push back on the port(s) that try to use it.

Based on what Mark found it looks like dldump() applies only to Solaris and was done specifically for emacs, so it's not worth the effort of carrying this feature in lld.