Skip to content

Commit 426906b

Browse files
committed
8210920: Native C++ tests are not using CXXFLAGS
Reviewed-by: mdoerr Backport-of: 481a045
1 parent cbe5a58 commit 426906b

File tree

4 files changed

+11
-23
lines changed

4 files changed

+11
-23
lines changed

make/autoconf/flags-cflags.m4

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -391,16 +391,6 @@ AC_DEFUN([FLAGS_SETUP_CFLAGS],
391391
CFLAGS="$CFLAGS_OLD"
392392
CXXFLAGS="$CXXFLAGS_OLD"
393393
394-
# Tests are only ever compiled for TARGET
395-
CFLAGS_TESTLIB="$CFLAGS_JDKLIB"
396-
CXXFLAGS_TESTLIB="$CXXFLAGS_JDKLIB"
397-
CFLAGS_TESTEXE="$CFLAGS_JDKEXE"
398-
CXXFLAGS_TESTEXE="$CXXFLAGS_JDKEXE"
399-
400-
AC_SUBST(CFLAGS_TESTLIB)
401-
AC_SUBST(CFLAGS_TESTEXE)
402-
AC_SUBST(CXXFLAGS_TESTLIB)
403-
AC_SUBST(CXXFLAGS_TESTEXE)
404394
])
405395

406396
################################################################################

make/autoconf/flags-ldflags.m4

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -51,9 +51,7 @@ AC_DEFUN([FLAGS_SETUP_LDFLAGS],
5151
5252
FLAGS_SETUP_LDFLAGS_CPU_DEP([BUILD], [OPENJDK_BUILD_])
5353
54-
LDFLAGS_TESTLIB="$LDFLAGS_JDKLIB"
55-
LDFLAGS_TESTEXE="$LDFLAGS_JDKEXE ${TARGET_LDFLAGS_JDK_LIBPATH}"
56-
AC_SUBST(LDFLAGS_TESTLIB)
54+
LDFLAGS_TESTEXE="${TARGET_LDFLAGS_JDK_LIBPATH}"
5755
AC_SUBST(LDFLAGS_TESTEXE)
5856
])
5957

make/autoconf/spec.gmk.in

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -508,11 +508,6 @@ LDCXX:=@FIXPATH@ @LDCXX@
508508
LIBCXX:=@LIBCXX@
509509

510510
# Compiler and linker flags used when building native tests
511-
CFLAGS_TESTLIB:=@CFLAGS_TESTLIB@
512-
CXXFLAGS_TESTLIB:=@CXXFLAGS_TESTLIB@
513-
CFLAGS_TESTEXE:=@CFLAGS_TESTEXE@
514-
CXXFLAGS_TESTEXE:=@CXXFLAGS_TESTEXE@
515-
LDFLAGS_TESTLIB:=@LDFLAGS_TESTLIB@
516511
LDFLAGS_TESTEXE:=@LDFLAGS_TESTEXE@
517512

518513
# BUILD_CC/BUILD_LD is a compiler/linker that generates code that is runnable on the

make/common/TestFilesCompilation.gmk

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -60,14 +60,16 @@ define SetupTestFilesCompilationBody
6060
ifeq ($$($1_TYPE), LIBRARY)
6161
$1_PREFIX = lib
6262
$1_OUTPUT_SUBDIR := lib
63-
$1_CFLAGS := $(CFLAGS_TESTLIB)
64-
$1_LDFLAGS := $(LDFLAGS_TESTLIB) $(call SET_SHARED_LIBRARY_ORIGIN)
63+
$1_BASE_CFLAGS := $(CFLAGS_JDKLIB)
64+
$1_BASE_CXXFLAGS := $(CXXFLAGS_JDKLIB)
65+
$1_LDFLAGS := $(LDFLAGS_JDKLIB) $(call SET_SHARED_LIBRARY_ORIGIN)
6566
$1_COMPILATION_TYPE := LIBRARY
6667
else ifeq ($$($1_TYPE), PROGRAM)
6768
$1_PREFIX = exe
6869
$1_OUTPUT_SUBDIR := bin
69-
$1_CFLAGS := $(CFLAGS_TESTEXE)
70-
$1_LDFLAGS := $(LDFLAGS_TESTEXE)
70+
$1_BASE_CFLAGS := $(CFLAGS_JDKEXE)
71+
$1_BASE_CXXFLAGS := $(CXXFLAGS_JDKEXE)
72+
$1_LDFLAGS := $(LDFLAGS_JDKEXE) $(LDFLAGS_TESTEXE)
7173
$1_COMPILATION_TYPE := EXECUTABLE
7274
else
7375
$$(error Unknown type: $$($1_TYPE))
@@ -91,8 +93,11 @@ define SetupTestFilesCompilationBody
9193
INCLUDE_FILES := $$(notdir $$(file)), \
9294
OBJECT_DIR := $$($1_OUTPUT_DIR)/support/$$(name), \
9395
OUTPUT_DIR := $$($1_OUTPUT_DIR)/$$($1_OUTPUT_SUBDIR), \
94-
CFLAGS := $$($1_CFLAGS) $$($1_CFLAGS_$$(name)), \
96+
CFLAGS := $$($1_BASE_CFLAGS) $$($1_CFLAGS) $$($1_CFLAGS_$$(name)), \
97+
CXXFLAGS := $$($1_BASE_CXXFLAGS) $$($1_CFLAGS) $$($1_CFLAGS_$$(name)), \
9598
LDFLAGS := $$($1_LDFLAGS) $$($1_LDFLAGS_$$(name)), \
99+
DISABLED_WARNINGS_CXX_solstudio := wvarhidenmem, \
100+
DISABLED_WARNINGS_CXX_gcc := format, \
96101
LIBS := $$($1_LIBS_$$(name)), \
97102
OPTIMIZATION := $$(if $$($1_OPTIMIZATION_$$(name)),$$($1_OPTIMIZATION_$$(name)),LOW), \
98103
COPY_DEBUG_SYMBOLS := false, \

0 commit comments

Comments
 (0)