[Bug 197577] [PATCH] devel/nspr: Has no mips/mips64 knowledge

[bugzilla-noreply at freebsd.org]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=197577

--- Comment #13 from Jan Beich <jbeich at 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.