2018-05-29 15:11 GMT+09:00 Christoph Hellwig <h...@infradead.org>: > On Mon, May 28, 2018 at 06:35:05PM +0200, Luc Van Oostenryck wrote: >> By default, sparse assumes a 64bit machine when compiled on x86-64 >> and 32bit when compiled on anything else. >> >> This can of course create all sort of problems when this doesn't >> correspond to the target's machine size, like issuing false >> warnings like: 'shift too big (32) for type unsigned long' or >> is 64bit while sparse was compiled on a 32bit machine, or worse, >> to not emit legitimate warnings. >> >> Fix this by passing the appropriate -m32/-m64 flag to sparse. > > Can we please move this to the common Kbuild code using the > CONFIG_64BIT syombol? This really should not need boiler plate in > every architecture.
I agree. Luc did so for -mbig/little-endian: https://patchwork.kernel.org/patch/10433957/ We should do likewise for -m32/64. -- Best Regards Masahiro Yamada
