Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 11 May 2015 18:16:26 +0000
From:      bugzilla-noreply@freebsd.org
To:        freebsd-mips@FreeBSD.org
Subject:   [Bug 197577] [PATCH] devel/nspr: Has no mips/mips64 knowledge
Message-ID:  <bug-197577-27794-7CYYUlVOkZ@https.bugs.freebsd.org/bugzilla/>
In-Reply-To: <bug-197577-27794@https.bugs.freebsd.org/bugzilla/>
References:  <bug-197577-27794@https.bugs.freebsd.org/bugzilla/>

next in thread | previous in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=197577

--- Comment #13 from Jan Beich <jbeich@FreeBSD.org> ---
Comment on attachment 156362
  --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=156362
patch to enable proper support on mips/mips64

>+#elif defined(__mips__)
>+#define _PR_SI_ARCHITECTURE "mips"
>+#elif defined(__mips64__)
>+#define _PR_SI_ARCHITECTURE "mips64"

__mips__ is defined for both in gcc and clang which means the first match wins
i.e., always 32bit.

  contrib/gcc/config/mips/freebsd.h
  contrib/llvm/tools/clang/lib/Basic/Targets.cpp.

I wonder if regression-test target would have caught it.

>+#elif defined(__mips__)
>+
>+#undef  IS_LITTLE_ENDIAN
>+#define IS_BIG_ENDIAN    1

See upstream reviewer comment. We may need to do smth similar to _linux.cfg.

-- 
You are receiving this mail because:
You are on the CC list for the bug.



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-197577-27794-7CYYUlVOkZ>