<div dir="ltr">Whew! But thanks for testing!</div><br><div class="gmail_quote"><div dir="ltr">On Tue, Mar 13, 2018 at 5:02 PM Ed Maste <<a href="mailto:emaste@freebsd.org">emaste@freebsd.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 13 March 2018 at 16:51, Ed Maste <<a href="mailto:emaste@freebsd.org" target="_blank">emaste@freebsd.org</a>> wrote:<br>
> On 7 February 2018 at 01:20, Chandler Carruth via llvm-dev<br>
> <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br>
>> I've landed the patch in r324449.<br>
><br>
> To confirm, this change *did not* make it into 6.0.0, correct? I'm<br>
> looking at adding the external thunk for FreeBSD now, and we have<br>
> 6.0.0 in HEAD. Assuming that future Clang releases (major, and<br>
> updates) will all use __x86_indirect_thunk_r11 we presumably want to<br>
> patch r324449 into the 6.0.0 in our tree.<br>
<br>
Dimitry points out it did get merged (r325082), so I must have been<br>
testing a version of 6.0.0 before the final release.<br>
<br>
Testing again now, with:<br>
FreeBSD clang version 6.0.0 (tags/RELEASE_600/final 326565) (based on<br>
LLVM 6.0.0)<br>
confirms it emits calls to __x86_indirect_thunk_r11; sorry for the false alarm.<br>
</blockquote></div>