From: Tejun Heo
> On Thu, Jun 12, 2014 at 08:46:44AM +0000, David Laight wrote:
> > I was slightly worried it might generate the boolean value - something
> > that you really don't want it to do.
>
> I don't think gcc is that stupid at this point.
>
> > I only looked at the output for the old version.
> > The compiler seemed to have converted:
> > if (preload)
> > x();
> > y;
> > if (preload)
> > z();
> > into:
> > if (preload) {
> > x(); y; z();
> > } else {
> > y;
> > }
> > and then found out that z() was empty, leaving two copies of y().
>
> So, nothing wrong, right?
Apart from the fact that it is badly optimised.
If z() were an empty #define rather than an empty inline function
then you'd end up with the:
if (preload)
z();
being completely optimised away.
David
--
To unsubscribe from this list: send the line "unsubscribe linux-sctp" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
