Re: [PATCH] let sparse warn on &inline_function

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, 20 May 2006 16:53:53 +0100, Derek M Jones wrote:
> 
> >The whole purpose of sparse is to warn on things that are
> >not invalid to do, but don't make much sense. ;)
> >It simply does not really make sense to mark something
> >as "inline" and take pointers to it.
> 
> It makes perfect sense for me to want 'direct' calls to be
> inlined and be willing to accept that calls via pointers
> will not be inlined.

Does this still make sense with CONFIG_CC_OPTIMIZE_FOR_SIZE set?

Imo this warning has a fair chance of catching things the developer
did not intend to write.  How to deal with things afterwards should -
as always - be up to the developer.

Jörn

-- 
Why do musicians compose symphonies and poets write poems?
They do it because life wouldn't have any meaning for them if they didn't.
That's why I draw cartoons.  It's my life.
-- Charles Shultz
-
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

[Index of Archives]     [Newbies FAQ]     [LKML]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Trinity Fuzzer Tool]

  Powered by Linux