On Thu, Dec 8, 2016 at 1:38 AM, Luc Van Oostenryck <luc.vanoostenryck@xxxxxxxxx> wrote: > The option -m64 to just set the size of longs & pointers to 64 bits. > > This patch makes this option much more effective by: > - adding define for LONG_MAX & SIZEOF_POINTER > - define __LP64__ I like this two change. > - let the dual option, -m32, do the same as -m64 but for 32 bit + max_int_alignment = 4; + bits_in_long = 32; This a feel a bit tricky is that, those value has its initial value in target.c. It is strange to overwrite it with the exact same value. It will have the effect change to the init value in target.c has no impact on the later part of the code. Maybe remove the init value in target.c and comment it is overwrite during the -m64 finalize if we do it this way. Chris -- 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
