From: kevans <kevans@FreeBSD.org>
Date: Wed, 18 Sep 2019 01:58:56 +0000 (+0000)
Subject: googletest: default-disable on all of MIPS for now
X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=bf614a9a6b76b43ea816f3965f2cee56f79aefe2;p=freebsd.git

googletest: default-disable on all of MIPS for now

Parts of the fusefs tests trigger a bug in current versions of llvm: IR
representation of some routine for the MIPS targets is a function with a
large number of arguments. This then leads the compiler on an hour+ long
goose chase, which is OK if you build the current tree but less-so if you're
trying external toolchain or doing a universe build involving mips when it
eventually gets switched over to LLVM.

Better, accurate details can be found in LLVM PR43263.
---

diff --git a/share/mk/src.opts.mk b/share/mk/src.opts.mk
index 1ed38a2a076..fac7f8d019d 100644
--- a/share/mk/src.opts.mk
+++ b/share/mk/src.opts.mk
@@ -108,7 +108,6 @@ __DEFAULT_YES_OPTIONS = \
     GDB \
     GNU_DIFF \
     GNU_GREP \
-    GOOGLETEST \
     GPIO \
     HAST \
     HTML \
@@ -260,6 +259,15 @@ __TT=${TARGET}
 __TT=${MACHINE}
 .endif
 
+# Default GOOGLETEST to off for MIPS while LLVM PR 43263 is active.  Part
+# of the fusefs tests trigger excessively long compile times.  It does
+# eventually succeed, but this shouldn't be forced on those building by default.
+.if ${__TT} == "mips"
+__DEFAULT_NO_OPTIONS+=	GOOGLETEST
+.else
+__DEFAULT_YES_OPTIONS+=	GOOGLETEST
+.endif
+
 # All supported backends for LLVM_TARGET_XXX
 __LLVM_TARGETS= \
 		aarch64 \