<br><br><div class="gmail_quote">On Fri, Oct 16, 2009 at 8:48 AM, ron minnich <span dir="ltr"><<a href="mailto:rminnich@gmail.com">rminnich@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im">On Fri, Oct 16, 2009 at 7:41 AM, Myles Watson <<a href="mailto:mylesgw@gmail.com">mylesgw@gmail.com</a>> wrote:<br>
<br>
> The problem with this style is that it doesn't support conditional<br>
> compilation.  In this case it doesn't matter, but if you wanted to put in:<br>
><br>
> if(CONFIG_HAVE_MP_TABLE) {<br>
>      use_some_MP_defines();<br>
> }<br>
><br>
> It would break, wouldn't it?  Since CONFIG_ variables are used to control<br>
> what gets included in the image, I think we should stick with #if.<br>
<br>
</div>Why would that break?<font color="#888888"><br></font></blockquote><div>Before it gets optimized out it will look for the definition of the function which wasn't compiled in and whose headers weren't included.  Won't it break?  I could try it with some of the k8 raminit code that has conditional includes all over the place.<br>
<br>Thanks,<br>Myles <br></div></div><br>