Date: Tue, 12 Mar 2013 08:52:09 +0100 From: Dimitry Andric <dim@freebsd.org> To: Adrian Chadd <adrian@freebsd.org> Cc: freebsd-hackers@freebsd.org, Joerg Sonnenberger <joerg@britannica.bec.de> Subject: Re: clang - odd macro / conditional expansion behaviour? Message-ID: <D4060110-61CB-49E3-9064-2A3F1FBAC27D@freebsd.org> In-Reply-To: <CAJ-Vmonn1m_AMiJbxfGDM%2BXrpfF%2BLKunSeoUWgkF9QVaMKf5ew@mail.gmail.com> References: <CAJ-Vmonn1m_AMiJbxfGDM%2BXrpfF%2BLKunSeoUWgkF9QVaMKf5ew@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Mar 12, 2013, at 04:17 , Adrian Chadd <adrian@freebsd.org> wrote: > In file included from > = /usr/home/adrian/work/freebsd/ath/head/src/sys/modules/ath/../../contrib/s= ys/dev/ath/ath_hal/ar9300/ar9300_eeprom.c:21: > = /usr/home/adrian/work/freebsd/ath/head/src/sys/modules/ath/../../contrib/s= ys/dev/ath/ath_hal/ar9300/ar9300template_generic.h:107:3: > error: implicit conversion from 'int' to > 'u_int8_t' (aka 'unsigned char') changes value from -477 to 35 > [-Werror,-Wconstant-conversion] > FREQ2FBIN(2412, 1), > ^~~~~~~~~~~~~~~~~~ > = /usr/home/adrian/work/freebsd/ath/head/src/sys/modules/ath/../../contrib/s= ys/dev/ath/ath_hal/ar9300/ar9300eep.h:136:65: > note: expanded from macro 'FREQ2FBIN' > (((y) =3D=3D HAL_FREQ_BAND_2GHZ) ? ((x) - 2300) : (((x) - 4800) / = 5)) > ~~~~~~~~~~~~~^~~ I cannot find the exact code you are referencing here, but I assume it = is some sort of global initialization? If so, this is most likely = <http://llvm.org/bugs/show_bug.cgi?id=3D10030>, which has been = languishing in LLVM's Bugzilla for a way too long time. :-(
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?D4060110-61CB-49E3-9064-2A3F1FBAC27D>