From bf614a9a6b76b43ea816f3965f2cee56f79aefe2 Mon Sep 17 00:00:00 2001 From: kevans Date: Wed, 18 Sep 2019 01:58:56 +0000 Subject: [PATCH] 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. --- share/mk/src.opts.mk | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) 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 \ -- 2.39.5