Pass 0 to __builtin_frame_address() to appease modern GCC.
ClosedPublic
Actions

Authored by jhb on Nov 28 2019, 8:35 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Jun 6, 3:27 AM

	Unknown Object (File)
	Sat, Jun 6, 1:32 AM

	Unknown Object (File)
	Sat, Jun 6, 1:26 AM

	Unknown Object (File)
	Thu, Jun 4, 4:50 AM

	Unknown Object (File)
	Sat, May 23, 9:20 AM

	Unknown Object (File)
	Sat, May 23, 9:19 AM

	Unknown Object (File)
	Sat, May 23, 2:51 AM

	Unknown Object (File)
	Sat, May 23, 2:51 AM

View All Files

Subscribers

imp

Details

Reviewers

marius
imp

Commits

rS355374: Pass 0 to __builtin_frame_address() to appease modern GCC.

Summary

Modern versions of GCC warn about passing non-zero values to
__builtin_frame_address(). Passing 1 is a cosmetic change to
remove the db_trace_self() frame from the printed stack trace.

Test Plan

lets sparc64 GENERIC build with GCC 6.x.
compile tested only, not run tested

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jhb created this revision.Nov 28 2019, 8:35 PM

Herald added a subscriber: imp. · View Herald TranscriptNov 28 2019, 8:35 PM

Harbormaster completed remote builds in B27843: Diff 65024.Nov 28 2019, 8:35 PM

imp accepted this revision.Nov 28 2019, 8:44 PM

This revision is now accepted and ready to land.Nov 28 2019, 8:44 PM

looks good to me

Closed by commit rS355374: Pass 0 to __builtin_frame_address() to appease modern GCC. (authored by jhb). · Explain WhyDec 3 2019, 11:20 PM

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rS355374: Pass 0 to __builtin_frame_address() to appease modern GCC..

Revision Contents
Changeset List

Path

Size

head/

sys/

sparc64/

db_trace.c

2 lines

Diff 65190

View Options

head/sys/sparc64/sparc64/db_trace.c

Pass 0 to __builtin_frame_address() to appease modern GCC.ClosedPublicActions