Page MenuHomeFreeBSD
Differential D44877

libgcc_s: restore __*_frame_info symbols post llvm18
ClosedPublic
Actions

Authored by brooks on Fri, Apr 19, 9:28 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, May 8, 6:07 PM
Unknown Object (File)
Fri, May 3, 3:57 AM
Unknown Object (File)
Tue, Apr 30, 11:02 AM
Unknown Object (File)
Tue, Apr 30, 10:56 AM
Unknown Object (File)
Sun, Apr 28, 11:51 AM
Unknown Object (File)
Fri, Apr 26, 3:13 PM
Unknown Object (File)
Fri, Apr 26, 3:13 PM
Unknown Object (File)
Fri, Apr 26, 3:13 PM
View All 13 Files
Subscribers
imp

Details

Reviewers
dim
Commits
rG82d22a88c340: libgcc_{eh,s}: restore __*_frame_info symbols post llvm18
rGa4f6bfe30406: libgcc_{eh,s}: restore __*_frame_info symbols post llvm18
rG0b9e3585805e: libgcc_{eh,s}: restore __*_frame_info symbols post llvm18
Summary

The upstream llvm commit 5eb44df1b64d made the addition of these GCC
compatability symbols dependent on build configuration rather than
hardcoded for amd64, i386, and powerpc. Reenable them.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 57235
Build 54123: arc lint + arc unit

Event Timeline

brooks created this revision.Fri, Apr 19, 9:28 PM
Herald added a subscriber: imp. · View Herald TranscriptFri, Apr 19, 9:28 PM
brooks requested review of this revision.Fri, Apr 19, 9:28 PM
Harbormaster completed remote builds in B57235: Diff 137418.Fri, Apr 19, 9:28 PM
brooks added a child revision: D44216: Make WITHOUT_UNDEFINED_VERSION the default.Fri, Apr 19, 9:31 PM
dim accepted this revision.Fri, Apr 19, 9:35 PM

Otherwise, LGTM.

lib/libgcc_eh/Makefile.inc
20

I'm always a bit confused by these unquoted strings... But apparently make groks this? :)

This revision is now accepted and ready to land.Fri, Apr 19, 9:35 PM
dim added a comment.Fri, Apr 19, 9:36 PM

Also, should we MFC this?

brooks added inline comments.Fri, Apr 19, 9:42 PM
lib/libgcc_eh/Makefile.inc
20

I can quote them if you prefer

dim added inline comments.Fri, Apr 19, 9:48 PM
lib/libgcc_eh/Makefile.inc
20

I can quote them if you prefer

Please do, other string values are also quoted in this same file.

Closed by commit rG0b9e3585805e: libgcc_{eh,s}: restore __*_frame_info symbols post llvm18 (authored by brooks). · Explain WhyFri, Apr 19, 9:56 PM
This revision was automatically updated to reflect the committed changes.
brooks added a commit: rG0b9e3585805e: libgcc_{eh,s}: restore __*_frame_info symbols post llvm18.
dim added a commit: rGa4f6bfe30406: libgcc_{eh,s}: restore __*_frame_info symbols post llvm18.Sat, Apr 20, 10:33 AM
dim added a commit: rG82d22a88c340: libgcc_{eh,s}: restore __*_frame_info symbols post llvm18.

Revision Contents
Changeset List

PathSize
lib/
libgcc_eh/
3 lines

Diff 137418

View Options

lib/libgcc_eh/Makefile.inc

Loading...