Date: Sat, 27 Oct 2012 12:07:20 +0200 From: Ed Schouten <ed@80386.nl> To: Roman Divacky <rdivacky@freebsd.org> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r242080 - in head/contrib/llvm/tools/clang: include/clang/Basic lib/Sema Message-ID: <CAJOYFBBoCy3bCuLBbJcWPkTjLXk_rpPpQPZ=xP8n2hi3xss2Lg@mail.gmail.com> In-Reply-To: <20121025121223.GA23573@freebsd.org> References: <201210251013.q9PADwYO029047@svn.freebsd.org> <20121025121223.GA23573@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Hi Roman, 2012/10/25 Roman Divacky <rdivacky@freebsd.org>: > Why? I can understand cherry picking revisions that fix bugs FreeBSD > users hit but why additional features? Because I discussed this specific warning/compiler flag with others at EuroBSDCon and most people seemed interested. By having it in our compiler binary, it lowers the barrier for people to try it on their code. Also, I am considering adding this to WARNS=6 (or maybe a WARNS=7?) by the time we make Clang the default on amd64/i386. I discussed it with the respective maintainer (dim@) and he approved, so I don't see the problem. -- Ed Schouten <ed@80386.nl>
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAJOYFBBoCy3bCuLBbJcWPkTjLXk_rpPpQPZ=xP8n2hi3xss2Lg>