On Wed, Oct 18, 2017 at 11:26 PM, Christopher Li <sparse@xxxxxxxxxxx> wrote: > Hi, > > This is the V4 version of the debug build change. > > https://git.kernel.org/pub/scm/devel/sparse/chrisl/sparse.git/log/?h=debug-target-v4 > > This change allow sparse to build debug version of sparse > along side of the release version. The debug build of sparse > can be used by patches like ptrlist ref count to check the nest > loop modify usage. The debug build can add other verification feature > that might slow down sparse. > > I plan to merge that to master. If there is other objections, it is > a good time to raise it now. No real objections but I have some doubts about all the debug part. I think that people hacking on sparse and who need to debug know what they need and want to debug. For example, the OPT=0 is, IMO, useless as you generally need others flags too. Also, when you debug, you generally need to rerun things several times, so using extra options on the command line is not ideal (a mechanism like local.mk is better suited but local.mk itself is not). Also, what's the real need for dbgbuild/ & debug/ ? IMO, it's a big complexification of all the rules with plenty of duplicated things for very few, if any, benefits. -- Luc -- To unsubscribe from this list: send the line "unsubscribe linux-sparse" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
