<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Aug 17, 2018 at 1:27 PM, Dimitry Andric <span dir="ltr"><<a href="mailto:dimitry@andric.com" target="_blank">dimitry@andric.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 16 Aug 2018, at 00:51, Joerg Sonnenberger via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> wrote:<br>
<span class="gmail-">> On Tue, Aug 07, 2018 at 09:49:16PM +0200, Dimitry Andric via llvm-dev wrote:<br>
>> This is a regression caused by <a href="https://reviews.llvm.org/rL323281" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>rL323281</a>:<br>
>> <br>
>> ------------------------------<wbr>------------------------------<wbr>------------<br>
>> r323281 | wmi | 2018-01-23 23:27:57 +0000 (Tue, 23 Jan 2018) | 12 lines<br>
>> <br>
>> Adjust MaxAtomicInlineWidth for i386/i486 targets.<br>
>> <br>
>> This is to fix the bug reported in <a href="https://bugs.llvm.org/show_bug.cgi?id=34347#c6" rel="noreferrer" target="_blank">https://bugs.llvm.org/show_<wbr>bug.cgi?id=34347#c6</a>.<br>
>> Currently, all  MaxAtomicInlineWidth of x86-32 targets are set to 64. However,<br>
>> i386 doesn't support any cmpxchg related instructions. i486 only supports cmpxchg.<br>
>> So in this patch MaxAtomicInlineWidth is reset as follows:<br>
>> For i386, the MaxAtomicInlineWidth should be 0 because no cmpxchg is supported.<br>
>> For i486, the MaxAtomicInlineWidth should be 32 because it supports cmpxchg.<br>
>> For others 32 bits x86 cpu, the MaxAtomicInlineWidth should be 64 because of cmpxchg8b.<br>
> <br>
</span>> This seems to be somewhat undesirable. Does *anyone* care about real<br>
> i386 support at this point? NetBSD certainly doesn't and I think we are<br>
> already the odd man for a number of cases like this.<br>
<br>
<br>
Yes, and since this causes quite a number of regressions for us, I would<br>
really prefer this revision to be reverted, at least in the 7.0.0<br>
branch.  I have already reverted it locally in our FreeBSD project<br>
branch for importing llvm/clang 7.0.0.  Hans, what is your opinion about<br>
this?<br>
<span class="gmail-HOEnZb"><font color="#888888"><br>
-Dimitry<br>
<br></font></span></blockquote><div> </div></div>Sorry I missed the thread for quite a while. Dimitry, I am very confused because you reported the issue in <a href="https://bugs.llvm.org/show_bug.cgi?id=34347#c6">https://bugs.llvm.org/show_bug.cgi?id=34347#c6</a>, so you want r323281 to be reverted and let llvm to generate cmxchg8b instruction for i486?</div><div class="gmail_extra"><br></div><div class="gmail_extra">Wei.</div><div class="gmail_extra"><br></div></div>