<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Feb 24, 2015 at 4:41 PM, Filip Pizlo <span dir="ltr"><<a href="mailto:fpizlo@apple.com" target="_blank">fpizlo@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">> On Feb 24, 2015, at 4:39 PM, Hans Wennborg <<a href="mailto:hans@chromium.org">hans@chromium.org</a>> wrote:<br>
><br>
> On Tue, Feb 24, 2015 at 3:46 PM, Filip Pizlo <<a href="mailto:fpizlo@apple.com">fpizlo@apple.com</a>> wrote:<br>
>> Proposed fix:<br>
><br>
>> -                         unsigned Unused, char **OutMessages) {<br>
>> +                         LLVMLinkerMode Unused, char **OutMessages) {<br>
><br>
> Do we really need to use the enum in the function prototype? Peter's<br>
> message suggests this might require a bindings update. The enumerators<br>
> should convert to unsigned anyway, right?<br>
<br>
</span>WebKit points a pointer at this function, so the signature needs to stay the same.</blockquote></div><br>It's a C function, so its really not clear why this should work without source changes, and it should clearly work with source changes.</div></div>