Page MenuHomeFreeBSD

Reduce Duplication in sys/*/include files
AcceptedPublic

Authored by pnagato_protonmail.com on Sep 2 2020, 6:48 AM.

Details

Reviewers
imp
Summary

@imp Opening a diff for feedback/discussion

Diff Detail

Repository
rS FreeBSD src repository
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 33348
Build 30658: arc lint + arc unit

Event Timeline

Added the duplication to a new file

Would it make more sense to create sys/x86/include/x86_ieeefp.h and include it in amd64/include/ieeefp.h and i386/include/ieeefp.h?

BTW, please consider submitting/updating patch with arc tool: https://wiki.freebsd.org/Phabricator

Move duplicated code under x86/include

@lwhsu Thank you, i moved the duplicated code under x86/include/x86_ieeefp.h and i was able to arc to update the diff.

Add new line and space after #include

@imp : ping, do you think this makes sense? It looks good to me but I'm not sure the convention is naming it x86_ieeefp.h or _ieeefp.h, I saw both under x86/include.

I think this is fine

This revision is now accepted and ready to land.Sep 12 2020, 3:08 PM
pnagato_protonmail.com retitled this revision from WIP: Reduce Duplication in sys/*/include files to Reduce Duplication in sys/*/include files.Sep 16 2020, 3:50 AM

One final question, is the "Deprecated historical FPU control interface" comment also applicable to the remaining codes in amd64 and i386?