Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 16 Feb 2023 16:54:47 GMT
From:      Warner Losh <imp@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org
Subject:   git: a86b9ed22ece - stable/13 - powerpc: Drop old workaround for old gcc
Message-ID:  <202302161654.31GGslBY085708@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch stable/13 has been updated by imp:

URL: https://cgit.FreeBSD.org/src/commit/?id=a86b9ed22ece362331f44a933bbf5426a06f527a

commit a86b9ed22ece362331f44a933bbf5426a06f527a
Author:     Warner Losh <imp@FreeBSD.org>
AuthorDate: 2023-02-16 16:36:03 +0000
Commit:     Warner Losh <imp@FreeBSD.org>
CommitDate: 2023-02-16 16:52:06 +0000

    powerpc: Drop old workaround for old gcc
    
    Very old versions of gcc defined _BIG_ENDIAN and _LITTLE_ENDIAN. So to
    work around that, we undefined them here. However, that causes problems
    for programs that do:
    
    (and many other variations on that theme). Since this often is the
    result of weirdly nested includes in the ports world that are hard to
    unwind, drop this workaround to help more ports build out of the box.
    If there's still an issue here (and my testing hasn't shown it), we'll
    fix the issue in a brand-new way once I have a reproducer.
    
    This fixes the mesa-devel build, and others
    
    Sponsored by:           Netflix
    Tested by:              pkubaj
    MFC After:              3 days
    Differential Revision:  https://reviews.freebsd.org/D38564
    
    (cherry picked from commit e35448a901aa56aa232efbc4fe7a73a9bdac3511)
---
 sys/powerpc/include/endian.h | 12 ------------
 1 file changed, 12 deletions(-)

diff --git a/sys/powerpc/include/endian.h b/sys/powerpc/include/endian.h
index 9e28237bfd24..0fac50361632 100644
--- a/sys/powerpc/include/endian.h
+++ b/sys/powerpc/include/endian.h
@@ -36,18 +36,6 @@
 #define	_MACHINE_ENDIAN_H_
 
 #include <sys/_types.h>
-
-/*
- * GCC defines _BIG_ENDIAN and _LITTLE_ENDIAN equal to __BIG_ENDIAN__
- * and __LITTLE_ENDIAN__ (resp).
- */
-#ifdef _BIG_ENDIAN
-#undef _BIG_ENDIAN
-#endif
-#ifdef _LITTLE_ENDIAN
-#undef _LITTLE_ENDIAN
-#endif
-
 #include <sys/_endian.h>
 
 #endif /* !_MACHINE_ENDIAN_H_ */



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202302161654.31GGslBY085708>