Skip to content

Commit 33ec92e

Browse files
committed
build: Remove unused component CFLAGS
A number of components AC_SUBSTed per-component CFLAGS, despite never setting or (in all but one case) using the variable in the component's Makefile.am. Remove this dead code. Signed-off-by: Brian Barrett <[email protected]>
1 parent 16fa266 commit 33ec92e

File tree

13 files changed

+12
-18
lines changed

13 files changed

+12
-18
lines changed

ompi/mca/coll/hcoll/configure.m4

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
# Copyright (c) 2011 Mellanox Technologies. All rights reserved.
55
# Copyright (c) 2015 Research Organization for Information Science
66
# and Technology (RIST). All rights reserved.
7+
# Copyright (c) 2022 Amazon.com, Inc. or its affiliates. All Rights reserved.
78
# $COPYRIGHT$
89
#
910
# Additional copyrights may follow
@@ -30,7 +31,6 @@ AC_DEFUN([MCA_ompi_coll_hcoll_CONFIG],[
3031
[$2])
3132

3233
# substitute in the things needed to build hcoll
33-
AC_SUBST([coll_hcoll_CFLAGS])
3434
AC_SUBST([coll_hcoll_CPPFLAGS])
3535
AC_SUBST([coll_hcoll_LDFLAGS])
3636
AC_SUBST([coll_hcoll_LIBS])

ompi/mca/coll/ucc/configure.m4

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,6 @@ AC_DEFUN([MCA_ompi_coll_ucc_CONFIG],[
3333
OPAL_SUMMARY_ADD([Miscellaneous], [Open UCC], [], [$coll_ucc_happy])
3434

3535
# substitute in the things needed to build ucc
36-
AC_SUBST([coll_ucc_CFLAGS])
3736
AC_SUBST([coll_ucc_CPPFLAGS])
3837
AC_SUBST([coll_ucc_LDFLAGS])
3938
AC_SUBST([coll_ucc_LIBS])

ompi/mca/mtl/psm2/configure.m4

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313
# Copyright (c) 2010 Cisco Systems, Inc. All rights reserved.
1414
# Copyright (c) 2013 Sandia National Laboratories. All rights reserved.
1515
# Copyright (c) 2014 Intel Corporation. All rights reserved.
16+
# Copyright (c) 2022 Amazon.com, Inc. or its affiliates. All Rights reserved.
1617
# $COPYRIGHT$
1718
#
1819
# Additional copyrights may follow
@@ -42,7 +43,6 @@ AC_DEFUN([MCA_ompi_mtl_psm2_CONFIG],[
4243
[$2])
4344

4445
# substitute in the things needed to build psm2
45-
AC_SUBST([mtl_psm2_CFLAGS])
4646
AC_SUBST([mtl_psm2_CPPFLAGS])
4747
AC_SUBST([mtl_psm2_LDFLAGS])
4848
AC_SUBST([mtl_psm2_LIBS])

ompi/mca/topo/treematch/configure.m4

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
# Copyright (c) 2015 Intel, Inc. All rights reserved.
1010
# Copyright (c) 2019 Research Organization for Information Science
1111
# and Technology (RIST). All rights reserved.
12-
# Copyright (c) 2020 Amazon.com, Inc. or its affiliates.
12+
# Copyright (c) 2020-2022 Amazon.com, Inc. or its affiliates. All Rights reserved.
1313
# All Rights reserved.
1414
# $COPYRIGHT$
1515
#
@@ -25,7 +25,6 @@ AC_DEFUN([MCA_ompi_topo_treematch_CONFIG], [
2525
OPAL_CONFIG_TREEMATCH([topo_treematch], [$1], [$2])
2626
AC_CONFIG_FILES([ompi/mca/topo/treematch/Makefile])
2727

28-
AC_SUBST([topo_treematch_CFLAGS])
2928
AC_SUBST([topo_treematch_CPPFLAGS])
3029
AC_SUBST([topo_treematch_LDFLAGS])
3130
AC_SUBST([topo_treematch_LIBS])

opal/mca/common/ucx/Makefile.am

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
#
22
# Copyright (c) 2018 Mellanox Technologies. All rights reserved.
3+
# Copyright (c) 2022 Amazon.com, Inc. or its affiliates. All Rights reserved.
34
# $COPYRIGHT$
45
#
56
# Additional copyrights may follow
@@ -53,8 +54,6 @@ endif
5354

5455
lib@OPAL_LIB_NAME@mca_common_ucx_la_SOURCES = \
5556
$(headers) $(sources)
56-
lib@OPAL_LIB_NAME@mca_common_ucx_la_CFLAGS = \
57-
$(common_ucx_CFLAGS)
5857
lib@OPAL_LIB_NAME@mca_common_ucx_la_CPPFLAGS = \
5958
$(common_ucx_CPPFLAGS)
6059
lib@OPAL_LIB_NAME@mca_common_ucx_la_LDFLAGS = \
@@ -66,8 +65,6 @@ lib@OPAL_LIB_NAME@mca_common_ucx_la_LIBADD = \
6665
$(OMPI_TOP_BUILDDIR)/opal/mca/memory/libmca_memory.la
6766
lib@OPAL_LIB_NAME@mca_common_ucx_noinst_la_SOURCES = \
6867
$(headers) $(sources)
69-
lib@OPAL_LIB_NAME@mca_common_ucx_noinst_la_CFLAGS = \
70-
$(common_ucx_CFLAGS)
7168
lib@OPAL_LIB_NAME@mca_common_ucx_noinst_la_CPPFLAGS = \
7269
$(common_ucx_CPPFLAGS)
7370
lib@OPAL_LIB_NAME@mca_common_ucx_noinst_la_LDFLAGS = \

opal/mca/common/ucx/configure.m4

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
# -*- shell-script -*-
22
#
33
# Copyright (c) 2018 Mellanox Technologies. All rights reserved.
4+
# Copyright (c) 2022 Amazon.com, Inc. or its affiliates. All Rights reserved.
45
# $COPYRIGHT$
56
#
67
# Additional copyrights may follow
@@ -24,9 +25,7 @@ AC_DEFUN([MCA_opal_common_ucx_CONFIG],[
2425
[$1],
2526
[$2])
2627

27-
2828
# substitute in the things needed to build common_ucx
29-
AC_SUBST([common_ucx_CFLAGS])
3029
AC_SUBST([common_ucx_CPPFLAGS])
3130
AC_SUBST([common_ucx_LDFLAGS])
3231
AC_SUBST([common_ucx_LIBS])

opal/mca/smsc/cma/configure.m4

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
# Copyright (c) 2015 Research Organization for Information Science
1010
# and Technology (RIST). All rights reserved.
1111
# Copyright (c) 2021 Google, LLC. All rights reserved.
12+
# Copyright (c) 2022 Amazon.com, Inc. or its affiliates. All Rights reserved.
1213
# $COPYRIGHT$
1314
#
1415
# Additional copyrights may follow
@@ -24,7 +25,6 @@ AC_DEFUN([MCA_opal_smsc_cma_CONFIG],[
2425

2526
OPAL_CHECK_CMA([smsc_cma], [AC_CHECK_HEADER([sys/prctl.h]) $1], [$2])
2627

27-
AC_SUBST([smsc_cma_CFLAGS])
2828
AC_SUBST([smsc_cma_CPPFLAGS])
2929
AC_SUBST([smsc_cma_LDFLAGS])
3030
AC_SUBST([smsc_cma_LIBS])

opal/mca/smsc/knem/configure.m4

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
# Copyright (c) 2015 Research Organization for Information Science
1010
# and Technology (RIST). All rights reserved.
1111
# Copyright (c) 2021 Google, LLC. All rights reserved.
12+
# Copyright (c) 2022 Amazon.com, Inc. or its affiliates. All Rights reserved.
1213
# $COPYRIGHT$
1314
#
1415
# Additional copyrights may follow
@@ -24,7 +25,6 @@ AC_DEFUN([MCA_opal_smsc_knem_CONFIG],[
2425

2526
OPAL_CHECK_KNEM([smsc_knem], [$1], [$2])
2627

27-
AC_SUBST([smsc_knem_CFLAGS])
2828
AC_SUBST([smsc_knem_CPPFLAGS])
2929
AC_SUBST([smsc_knem_LDFLAGS])
3030
AC_SUBST([smsc_knem_LIBS])

opal/mca/smsc/xpmem/configure.m4

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
# Copyright (c) 2015 Research Organization for Information Science
1010
# and Technology (RIST). All rights reserved.
1111
# Copyright (c) 2021 Google, LLC. All rights reserved.
12+
# Copyright (c) 2022 Amazon.com, Inc. or its affiliates. All Rights reserved.
1213
# $COPYRIGHT$
1314
#
1415
# Additional copyrights may follow
@@ -24,7 +25,6 @@ AC_DEFUN([MCA_opal_smsc_xpmem_CONFIG],[
2425

2526
OPAL_CHECK_XPMEM([smsc_xpmem], [$1], [$2])
2627

27-
AC_SUBST([smsc_xpmem_CFLAGS])
2828
AC_SUBST([smsc_xpmem_CPPFLAGS])
2929
AC_SUBST([smsc_xpmem_LDFLAGS])
3030
AC_SUBST([smsc_xpmem_LIBS])

oshmem/mca/atomic/ucx/configure.m4

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
/*
22
* Copyright (c) 2015 Mellanox Technologies, Inc.
33
* All rights reserved.
4+
* Copyright (c) 2022 Amazon.com, Inc. or its affiliates. All Rights reserved.
45
* $COPYRIGHT$
56
*
67
* Additional copyrights may follow
@@ -24,7 +25,6 @@ AC_DEFUN([MCA_oshmem_atomic_ucx_CONFIG],[
2425
2526
2627
# substitute in the things needed to build ucx
27-
AC_SUBST([atomic_ucx_CFLAGS])
2828
AC_SUBST([atomic_ucx_CPPFLAGS])
2929
AC_SUBST([atomic_ucx_LDFLAGS])
3030
AC_SUBST([atomic_ucx_LIBS])

0 commit comments

Comments
 (0)