<div dir="ltr">Could you please land it? I don't have write access.<div><br><div class="gmail_quote"><div dir="ltr">On Thu, Jul 13, 2017 at 8:09 PM Hal Finkel <<a href="mailto:hfinkel@anl.gov" target="_blank">hfinkel@anl.gov</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
On 07/13/2017 01:05 PM, Denys Zariaiev via Phabricator via llvm-commits<br>
wrote:<br>
> denzp added a comment.<br>
><br>
> In <a href="https://reviews.llvm.org/D34555#802254" rel="noreferrer" target="_blank">https://reviews.llvm.org/D34555#802254</a>, @tra wrote:<br>
><br>
>> In <a href="https://reviews.llvm.org/D34555#802008" rel="noreferrer" target="_blank">https://reviews.llvm.org/D34555#802008</a>, @denzp wrote:<br>
>><br>
>>> Is it safe just to change data layout at clang side? Won't it break backward compatibility?<br>
>><br>
>> Clang and LLVM sides of this change should land together as they have to be in sync.<br>
>><br>
>> Other than that, AFAICT it should be safe as it affects only i128 which is not currently supported, so there should be no backward compatibility issues. That's in theory. You may want to ask this question on cfe-dev@.<br>
><br>
> Looks like we can simply update Clang's data layout and it should not break anything: cfe-dev thread <<a href="http://lists.llvm.org/pipermail/cfe-dev/2017-July/054649.html" rel="noreferrer" target="_blank">http://lists.llvm.org/pipermail/cfe-dev/2017-July/054649.html</a>>.<br>
><br>
> Do I need to create a "differential" for the clang patch? I attached the diff to my previous comment here, and changes there are quite trivial.<br>
<br>
No, just go ahead and recommit along with the Clang change.<br>
<br>
  -Hal<br>
<br>
><br>
><br>
> Repository:<br>
>    rL LLVM<br>
><br>
> <a href="https://reviews.llvm.org/D34555" rel="noreferrer" target="_blank">https://reviews.llvm.org/D34555</a><br>
><br>
><br>
><br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
<br>
--<br>
Hal Finkel<br>
Lead, Compiler Technology and Programming Languages<br>
Leadership Computing Facility<br>
Argonne National Laboratory<br>
<br>
</blockquote></div></div></div>