On Wed, Jul 22, 2015 at 05:37:57PM -0700, josh@xxxxxxxxxxxxxxxx wrote: > [Side note: for some reason, your mail had your message ordered *after* > your attached diff, so replies quote the diff before the message.] > > On Thu, Jul 23, 2015 at 12:54:25AM +0200, Nicolai Stange wrote: > > My initial intent was to rework the current integer constant expression > > handling in order to allow for the recognition of constant subexpressions > > built up by means of __builtin_choose_expr(). Hence the first part. > > > > However, since I had to touch the whole constant expression handling > > code anyways, I decided to experimentally extend it to support > > arithmetic constant expressions and address constants as well. Hence > > the second part. > > > > Since the additional information on expressions obtained through the > > first two parts is rather pointless without making any use of it, I > > implemented part three, the checking of static storage duration > > objects' initializers for constness. > > This part is the reason why there is a 'RFC' tag in the subject. > > It is up to you to decide whether letting sparse check for C99 > > conformity is a valuable thing to have or whether being stricter than > > GCC is counter-productive/completely idiotic. > > I think it's absolutely a valuable thing to have. It may or may not be > the right *default* behavior, but having an appropriate -W option to > enable it would be a good start. > > I've seen kernel maintainers ask people to not rely on GCC's lax > enforcement of constant initializers. I also think it's a very valuable thing to have. After all, it's the raison d'etre of sparse to make stricter checks than the standard or GCC. But then I wonder what's must be done for things like GCC's builtins? Shouldn't, for example, __builtin_bswap32(..) always propagte the constantness of it's argument or it specifically this sort of things that are the target of this patch serie? Regards, 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