| commit | 38d1124ec5cd10e117e320e25a55305dca0899b6 | |
| author | Linus Torvalds <torvalds@linux-foundation.org> | |
| Thu, 3 Apr 2014 19:01:00 +0000 (3 12:01 -0700) | ||
| committer | Christopher Li <sparse@chrisli.org> | |
| Thu, 3 Apr 2014 19:03:52 +0000 (3 12:03 -0700) | ||
| tree | eec5762fff8de734a664877d51a08eb03ac0bc58 | treesnapshot (tar.gz zip) |
| parent | 42ebe4166c52a0c4cb8b50ab488374ac5c92ff2b | commitdiff |
Fix error at anoymous unions
Ok, this fixes the warning, but we seem to still mess up the actual
initializer. It looks like some later phase gets the offset wrong, so
when we lay things out in memory, we'll put things at offset zero
(which is right for your test-case, but not if there was something
before that anonymous union).
Regardless, that only matters for real code generation, not for using
sparse as a semantic checker, so this patch is correct and is an
improvement.
Reported-by: Hans Verkuil <hverkuil@xs4all.nl>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Christopher Li <sparse@chrisli.org>
Ok, this fixes the warning, but we seem to still mess up the actual
initializer. It looks like some later phase gets the offset wrong, so
when we lay things out in memory, we'll put things at offset zero
(which is right for your test-case, but not if there was something
before that anonymous union).
Regardless, that only matters for real code generation, not for using
sparse as a semantic checker, so this patch is correct and is an
improvement.
Reported-by: Hans Verkuil <hverkuil@xs4all.nl>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Christopher Li <sparse@chrisli.org>
| evaluate.c | diffblobblamehistory |