<div dir="auto">Sorry for off conversation question but what motivated this patch? Some apps or usecase which fails without this?</div><div class="gmail_extra"><br><div class="gmail_quote">On Jul 7, 2017 10:00 PM, "Artem Belevich via Phabricator via llvm-commits" <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">tra requested changes to this revision.<br>
tra added a comment.<br>
This revision now requires changes to proceed.<br>
<br>
In <a href="https://reviews.llvm.org/D34555#802008" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>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>
<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>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D34555" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D34555</a><br>
<br>
<br>
<br>
______________________________<wbr>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">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/<wbr>mailman/listinfo/llvm-commits</a><br>
</blockquote></div></div>