Messages in this thread | | | Date | Tue, 22 Nov 2011 09:42:40 +0100 | From | DM <> | Subject | Re: [PATCH 0/2] Stop some of the abuse of BUG() where compile time checks should be used. |
| |
On 2011-11-22 02:31, David Daney wrote: > From: David Daney <david.daney@cavium.com> > > After some, perhaps justified, reluctance to merge dummy symbol > definitions containing BUG() into header files, I propose these patches > instead. > > We define a new compile time assertion BUILD_BUG_ON_USED() that can be > used in places were we expect the compiler's dead code elimination to > get rid of code. This happens mostly in code dealing with huge pages, > but in other places as well. > > The first patch adds BUILD_BUG_ON_USED(), the second gets rid of one > of the main abusers of BUG(). >
Perhaps BUILD_BUG() is a more consistent name for this?
We would then have BUG() and BUG_ON(x) for run-time vs BUILD_BUG() and BUILD_BUG_ON(x) for compile-time.
/DM
| |