[lkml]   [2007]   [Jun]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH 16/16] fix handling of integer constant expressions
    Al Viro wrote:-

    > > See for more discussion. Yes it's an old
    > > bug...
    > Humm... Right, so __builtin_offsetof() needs special treatment too.
    > Oh, bugger. Is
    > offsetof(struct foo, a.x[n])
    > a documented extension? I _know_ that it's not promised by 7.17,
    > but gcc eats it (and obviously that sucker requires extra treatment
    > in that case).

    I asked on comp.std.c about this; the feeling was that only identifiers
    are intended to be permitted as the 2nd argument to offsetof. If true
    the standard has a very obscure way of stating something that could
    be said much more simply.

    > Parsing __builtin_offsetof() arguments is going to be fun ;-/ Right
    > now sparse has it as a predefined macro, but if we want to do that
    > kind of analysis, we need to really parse it. OTOH, that's not
    > such a big deal... Parser would need to accept
    > ident ( \[ expr \] | . ident )*

    don't forget -> if you're going to accept extra stuff. GCC forgot ->
    with the parser rewrite, yes I filed a PR.

    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2007-06-24 23:51    [W:0.030 / U:17.924 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site