Date: Sat, 2 Jun 2018 22:27:22 -0700 From: Mark Millard <marklmi@yahoo.com> To: shawn.webb@hardenedbsd.org, FreeBSD Current <freebsd-current@freebsd.org> Cc: mmacy@mattmacy.io Subject: Re: Weird conditional logic in lib/libpmc/Makefile Message-ID: <C3892FC0-DD53-404E-8F6E-0772F52FCB99@yahoo.com>
next in thread | raw e-mail | index | archive | help
Shawn Webb shawn.webb at hardenedbsd.org wrote on Sun Jun 3 01:08:19 UTC 2018 : > Below is some logic pasted from lib/libpmc/Makefile: > > .if ${MACHINE_ARCH} == "amd64" || ${MACHINE_ARCH} == "i386" > > .if ${MACHINE_ARCH} == "aarch64" > EVENT_ARCH="arm64" > .elif ${MACHINE_ARCH} == "amd64" || ${MACHINE_ARCH} == "i386" > EVENT_ARCH="x86" > .elif ${MACHINE_ARCH} == "powerpc" > EVENT_ARCH="powerpc" > .endif > > JEVENTS= ${BTOOLSPATH:U.}/pmu-events/jevents > # This file is built in a subdirectory so never try to rebuild > # it here due to missing meta file. > ${JEVENTS}: .NOMETA > > libpmc_events.c: ${JEVENTS} > ${JEVENTS} ${EVENT_ARCH} ${.CURDIR}/pmu-events/arch libpmc_events.c > SRCS+= libpmc_events.c > .endif > > Why perform the conditionals for aarch64 and powerpc when it can't be > those? "Am I missing something? > > I'm looping in Matt Macy, who was the last person to touch the file. It appears that every version of lib/libpmc/Makefile from head -r334242 and later has some variation of the odd nesting/value combinations that has "EVENT_ARCH" examples that are impossible to select as the code is structured. (The file version prior to -r334242 is from -r298107 .) === Mark Millard marklmi at yahoo.com ( dsl-only.net went away in early 2018-Mar)
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?C3892FC0-DD53-404E-8F6E-0772F52FCB99>