Page MenuHomeFreeBSD
Differential D36781

linux32: binutils as requires %eflags instead of %flags for CFI.
ClosedPublic
Actions

Authored by jhb on Sep 28 2022, 9:40 PM.
Tags
None
Referenced Files
F149933454: D36781.id.diff
Sat, Mar 28, 5:19 AM
Unknown Object (File)
Wed, Mar 25, 5:45 AM
Unknown Object (File)
Sun, Mar 22, 7:48 AM
Unknown Object (File)
Fri, Mar 20, 9:20 PM
Unknown Object (File)
Wed, Mar 18, 1:16 AM
Unknown Object (File)
Wed, Mar 18, 1:16 AM
Unknown Object (File)
Feb 12 2026, 3:49 AM
Unknown Object (File)
Jan 14 2026, 11:27 AM
View All 86 Files
Subscribers
dim
imp

Details

Reviewers
kib
Commits
rG20125b23ff15: linux32: binutils as requires %eflags instead of %flags for CFI.
rGa35572b16e38: linux32: binutils as requires %eflags instead of %flags for CFI.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 47593
Build 44480: arc lint + arc unit

Event Timeline

jhb created this revision.Sep 28 2022, 9:40 PM
Herald added a subscriber: imp. · View Herald TranscriptSep 28 2022, 9:40 PM
jhb requested review of this revision.Sep 28 2022, 9:40 PM
Harbormaster completed remote builds in B47593: Diff 111161.Sep 28 2022, 9:40 PM
kib accepted this revision.Sep 28 2022, 9:42 PM

Does clang work with %eflags?

This revision is now accepted and ready to land.Sep 28 2022, 9:42 PM
jhb added a comment.Sep 28 2022, 9:43 PM
In D36781#834878, @kib wrote:

Does clang work with %eflags?

No. :( (I tried). Arguably %eflags is more correct, and possibly it should be reported as a bug to LLVM upstream to use the eflags/rflags names.

jhb added a subscriber: dim.Sep 28 2022, 9:43 PM

(Adding dim@ in case he wants to report an upstream bug about supporting %eflags upstream)

Closed by commit rGa35572b16e38: linux32: binutils as requires %eflags instead of %flags for CFI. (authored by jhb). · Explain WhySep 29 2022, 10:08 PM
This revision was automatically updated to reflect the committed changes.
jhb added a commit: rGa35572b16e38: linux32: binutils as requires %eflags instead of %flags for CFI..
jhb added a commit: rG20125b23ff15: linux32: binutils as requires %eflags instead of %flags for CFI..Nov 11 2022, 6:38 PM

Revision Contents
Changeset List

PathSize
sys/
amd64/
linux32/
8 lines

Diff 111161

View Options

sys/amd64/linux32/linux32_locore.asm

Loading...