<br><br><div class="gmail_quote">On Thu, Mar 25, 2010 at 8:52 AM, Stefan Reinauer <span dir="ltr"><<a href="mailto:stepan@coresystems.de">stepan@coresystems.de</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><div></div><div class="h5">On 3/25/10 3:47 PM, Myles Watson wrote:<br>
>>> On 3/24/10 11:02 PM, repository service wrote:<br>
>>><br>
>>>> -extern unsigned char AmlCode[];<br>
>>>> +extern const acpi_header_t AmlCode;<br>
>>>><br>
>>> And we're positive, this always does the right thing with gcc?<br>
>>><br>
>> I am told that AmlCode is defined as array of (unsigned) char in<br>
>> some other file.  Declaring it as some other type here is not<br>
>> valid C, and *will* break with GCC, with some options (-combine<br>
>> or LTO at least) -- it will not compile.<br>
>><br>
> The biggest worry for me is incorrect execution.  If it doesn't compile when<br>
> it breaks, then that's a good thing.<br>
><br>
</div></div>Well, if it breaks with LTO, we should fix it right away... I'll try and<br>
prepare something.<font color="#888888"><br></font></blockquote><div>What do the IASL folks suggest?  Do they have a way of doing this "correctly"?<br><br>It seems like others must have this problem of needing to force a binary blob into a struct.<br>
<br>Thanks,<br>Myles <br></div></div><br>