Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 4 Jan 2020 16:31:43 +0000 (UTC)
From:      Tobias Kortkamp <tobik@FreeBSD.org>
To:        ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-branches@freebsd.org
Subject:   svn commit: r522043 - branches/2020Q1/games/openomf
Message-ID:  <202001041631.004GVhBc027316@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: tobik
Date: Sat Jan  4 16:31:43 2020
New Revision: 522043
URL: https://svnweb.freebsd.org/changeset/ports/522043

Log:
  MFH: r521983
  
  games/openomf: Unbreak build on i386 (broken in r511074)
  
  warning: unknown warning option '-Wno-unused-but-set-variable'; did you mean '-Wno-unused-const-variable'? [-Wunknown-warning-option]
  external/dumb/dumb/src/it/itrender.c:721:11: error: always_inline function '_mm_setzero_ps' requires target feature 'mmx', but would be inlined into function 'it_filter_sse' that is compiled without support for 'mmx'
                  temp1 = _mm_setzero_ps();
                          ^
  external/dumb/dumb/src/it/itrender.c:722:10: error: always_inline function '_mm_cvtsi32_ss' requires target feature 'mmx', but would be inlined into function 'it_filter_sse' that is compiled without support for 'mmx'
                  data = _mm_cvtsi32_ss( temp1, currsample );
                         ^
  external/dumb/dumb/src/it/itrender.c:723:11: error: always_inline function '_mm_cvtsi32_ss' requires target feature 'mmx', but would be inlined into function 'it_filter_sse' that is compiled without support for 'mmx'
                  temp2 = _mm_cvtsi32_ss( temp1, prevsample );
                          ^
  external/dumb/dumb/src/it/itrender.c:724:13: error: always_inline function '_mm_loadu_ps' requires target feature 'mmx', but would be inlined into function 'it_filter_sse' that is compiled without support for 'mmx'
                  impulse = _mm_loadu_ps( (const float *) &imp );
                            ^
  external/dumb/dumb/src/it/itrender.c:728:12: error: always_inline function '_mm_cvtsi32_ss' requires target feature 'mmx', but would be inlined into function 'it_filter_sse' that is compiled without support for 'mmx'
                          temp1 = _mm_cvtsi32_ss( data, src [i] );
                                  ^
  external/dumb/dumb/src/it/itrender.c:729:12: error: always_inline function '_mm_mul_ps' requires target feature 'mmx', but would be inlined into function 'it_filter_sse' that is compiled without support for 'mmx'
                          temp1 = _mm_mul_ps( temp1, impulse );
                                  ^
  external/dumb/dumb/src/it/itrender.c:730:12: error: always_inline function '_mm_movehl_ps' requires target feature 'mmx', but would be inlined into function 'it_filter_sse' that is compiled without support for 'mmx'
                          temp2 = _mm_movehl_ps( temp2, temp1 );
                                  ^
  external/dumb/dumb/src/it/itrender.c:731:12: error: always_inline function '_mm_add_ps' requires target feature 'mmx', but would be inlined into function 'it_filter_sse' that is compiled without support for 'mmx'
                          temp1 = _mm_add_ps( temp1, temp2 );
                                  ^
  external/dumb/dumb/src/it/itrender.c:734:12: error: always_inline function '_mm_add_ps' requires target feature 'mmx', but would be inlined into function 'it_filter_sse' that is compiled without support for 'mmx'
                          temp1 = _mm_add_ps( temp1, temp2 );
                                  ^
  external/dumb/dumb/src/it/itrender.c:737:15: error: always_inline function '_mm_cvtss_si32' requires target feature 'mmx', but would be inlined into function 'it_filter_sse' that is compiled without support for 'mmx'
                          dst [i] += _mm_cvtss_si32( temp1 );
                                     ^
  external/dumb/dumb/src/it/itrender.c:740:16: error: always_inline function '_mm_cvtss_si32' requires target feature 'mmx', but would be inlined into function 'it_filter_sse' that is compiled without support for 'mmx'
                  currsample = _mm_cvtss_si32( temp1 );
                               ^
  external/dumb/dumb/src/it/itrender.c:742:16: error: always_inline function '_mm_cvtss_si32' requires target feature 'mmx', but would be inlined into function 'it_filter_sse' that is compiled without support for 'mmx'
                  prevsample = _mm_cvtss_si32( temp1 );
                               ^
  1 warning and 12 errors generated.
  
  http://beefy10.nyi.freebsd.org/data/113i386-default/521788/logs/errors/openomf-0.6.5_6.log
  http://beefy5.nyi.freebsd.org/data/120i386-default/521788/logs/errors/openomf-0.6.5_6.log
  
  Approved by:	ports-secteam blanket

Modified:
  branches/2020Q1/games/openomf/Makefile
Directory Properties:
  branches/2020Q1/   (props changed)

Modified: branches/2020Q1/games/openomf/Makefile
==============================================================================
--- branches/2020Q1/games/openomf/Makefile	Sat Jan  4 16:31:30 2020	(r522042)
+++ branches/2020Q1/games/openomf/Makefile	Sat Jan  4 16:31:43 2020	(r522043)
@@ -33,6 +33,8 @@ DESKTOP_ENTRIES="OpenOMF" "" \
 		"${PREFIX}/share/openomf/openomf_icon.png " \
 		"openomf" "Game;ArcadeGame;" false
 
+.include <bsd.port.options.mk>
+
 post-extract:
 	${RMDIR} ${WRKSRC}/external/libShadowDive
 	${LN} -s ${WRKSRC_shadow} ${WRKSRC}/external/libShadowDive
@@ -43,8 +45,10 @@ post-extract:
 post-patch:
 	${REINPLACE_CMD} -e 's,%%PREFIX%%,"${PREFIX}",g' \
 		${WRKSRC}/src/resources/pathmanager.c
+.if ${ARCH} != i386
 	${REINPLACE_CMD} -e 's,-msse ,,' \
 		${WRKSRC}/external/dumb/dumb/cmake/CMakeLists.txt
+.endif
 
 do-install:
 	${INSTALL_PROGRAM} ${WRKSRC}/openomf ${STAGEDIR}${PREFIX}/bin



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