On Mon, Feb 27, 2017 at 04:32:50PM +0800, Christopher Li wrote: > On Mon, Feb 27, 2017 at 4:22 PM, Christopher Li <sparse@xxxxxxxxxxx> wrote: > > On Fri, Feb 17, 2017 at 10:44 AM, Luc Van Oostenryck > > <luc.vanoostenryck@xxxxxxxxx> wrote: > >> Loading a bitfield correctly take in account the offset > >> of the bitfield inside the whole container integer. > >> But truncating it to the width of the bitfield is not done > >> or is done very implicitely (because the correct size is not lost). > > > >> Fix this by adding the missing truncating cast. > >> This will then gives something like: > >> get__bfu_a: > >> cast.3 %r2 <- (32) %arg1 > >> cast.32 %r3 <- (3) %r2 > >> ret.32 %r3 > > > > The patch looks good. Applied to sparse-next. > > Actually, after apply this patch. The testsuite failed > on matching the test case of the bitfield-size.c. Sorry for that. I changed my mind about the cast vs. and-mask and forgot to update the test case. Updated patch is coming. Luc Van Oostenryck -- 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
